|
Description
Martin Renvoize (ashimema)
2023-11-16 11:23:52 UTC
Created attachment 159059 [details] [review] Bug 35348: Add cookies link to opac footer This patch adds a link to prompt the cookies info dialogue into the opac footer when the CookieConsent feature is enabled. Work in progress here.. still more work to do and a test plan to follow Created attachment 159078 [details] [review] Bug 35348: Add cookies link to opac footer This patch adds a link that opens the cookies info dialogue to the opac footer when the CookieConsent feature is enabled. Test plan 1) Enable CookieConsent 2) On the OPAC you should now see a 'Cookies' link at the bottom of the page. 3) Clicking the link should open the "Cookie consent" modal where you can see the details of the cookies you have accepted and change your choices. 4) Bonus points: Enable 'OpacProblemReports' and note it aligns left of 'Cookies' in the same footer bar. Created attachment 159079 [details] [review] Bug 35348: Ensure users can access cookie details in staff client We remove the check on JSConsents content so that when cookie consent is enabled we retain the option for users to get more information about what cookies are being used. Test plan 1) Enable CookieConsents 2) Do not add any tracking cookies for the staff client into CookieConsentedJS preference 3) Note that 'My consents' appears in the user menu 4) Clicking 'My consents' opens the 'Cookie consent' modal displaying information about the cookies used. Created attachment 159080 [details] [review] Bug 35348: Remove cookie consents link from opac consents The cookies details modal is now visible for all users from a footer link, we no longer need it under the 'Consents' tab for logged in users. Test plan 1) Enable CookieConsent and add a placholder tracker cookie to CookieConsentedJS preference 2) Log into the OPAC 3) Note that the 'Consents' link no longer appears in the users page. 4) Enable 'PrivacyPolicyConsent' 5) Note that the 'Consents' link now appears but on navigating to it there is no link for Cookie consents. Created attachment 159091 [details] [review] Bug 35348: Add cookies link to opac footer This patch adds a link that opens the cookies info dialogue to the opac footer when the CookieConsent feature is enabled. Test plan 1) Enable CookieConsent 2) On the OPAC you should now see a 'Cookies' link at the bottom of the page. 3) Clicking the link should open the "Cookie consent" modal where you can see the details of the cookies you have accepted and change your choices. 4) Bonus points: Enable 'OpacProblemReports' and note it aligns left of 'Cookies' in the same footer bar. Created attachment 159092 [details] [review] Bug 35348: Ensure users can access cookie details in staff client We remove the check on JSConsents content so that when cookie consent is enabled we retain the option for users to get more information about what cookies are being used. Test plan 1) Enable CookieConsents 2) Do not add any tracking cookies for the staff client into CookieConsentedJS preference 3) Note that 'My consents' appears in the user menu 4) Clicking 'My consents' opens the 'Cookie consent' modal displaying information about the cookies used. Created attachment 159093 [details] [review] Bug 35348: Remove cookie consents link from opac consents The cookies details modal is now visible for all users from a footer link, we no longer need it under the 'Consents' tab for logged in users. Test plan 1) Enable CookieConsent and add a placholder tracker cookie to CookieConsentedJS preference 2) Log into the OPAC 3) Note that the 'Consents' link no longer appears in the users page. 4) Enable 'PrivacyPolicyConsent' 5) Note that the 'Consents' link now appears but on navigating to it there is no link for Cookie consents. Created attachment 159444 [details] [review] Bug 35348: Add cookies link to opac footer This patch adds a link that opens the cookies info dialogue to the opac footer when the CookieConsent feature is enabled. Test plan 1) Enable CookieConsent 2) On the OPAC you should now see a 'Cookies' link at the bottom of the page. 3) Clicking the link should open the "Cookie consent" modal where you can see the details of the cookies you have accepted and change your choices. 4) Bonus points: Enable 'OpacProblemReports' and note it aligns left of 'Cookies' in the same footer bar. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Created attachment 159445 [details] [review] Bug 35348: Ensure users can access cookie details in staff client We remove the check on JSConsents content so that when cookie consent is enabled we retain the option for users to get more information about what cookies are being used. Test plan 1) Enable CookieConsents 2) Do not add any tracking cookies for the staff client into CookieConsentedJS preference 3) Note that 'My consents' appears in the user menu 4) Clicking 'My consents' opens the 'Cookie consent' modal displaying information about the cookies used. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Created attachment 159446 [details] [review] Bug 35348: Remove cookie consents link from opac consents The cookies details modal is now visible for all users from a footer link, we no longer need it under the 'Consents' tab for logged in users. Test plan 1) Enable CookieConsent and add a placholder tracker cookie to CookieConsentedJS preference 2) Log into the OPAC 3) Note that the 'Consents' link no longer appears in the users page. 4) Enable 'PrivacyPolicyConsent' 5) Note that the 'Consents' link now appears but on navigating to it there is no link for Cookie consents. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Created attachment 165568 [details] [review] Bug 35348: Add cookies link to opac footer This patch adds a link that opens the cookies info dialogue to the opac footer when the CookieConsent feature is enabled. Test plan 1) Enable CookieConsent 2) On the OPAC you should now see a 'Cookies' link at the bottom of the page. 3) Clicking the link should open the "Cookie consent" modal where you can see the details of the cookies you have accepted and change your choices. 4) Bonus points: Enable 'OpacProblemReports' and note it aligns left of 'Cookies' in the same footer bar. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Created attachment 165569 [details] [review] Bug 35348: Ensure users can access cookie details in staff client We remove the check on JSConsents content so that when cookie consent is enabled we retain the option for users to get more information about what cookies are being used. Test plan 1) Enable CookieConsents 2) Do not add any tracking cookies for the staff client into CookieConsentedJS preference 3) Note that 'My consents' appears in the user menu 4) Clicking 'My consents' opens the 'Cookie consent' modal displaying information about the cookies used. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Created attachment 165570 [details] [review] Bug 35348: Remove cookie consents link from opac consents The cookies details modal is now visible for all users from a footer link, we no longer need it under the 'Consents' tab for logged in users. Test plan 1) Enable CookieConsent and add a placholder tracker cookie to CookieConsentedJS preference 2) Log into the OPAC 3) Note that the 'Consents' link no longer appears in the users page. 4) Enable 'PrivacyPolicyConsent' 5) Note that the 'Consents' link now appears but on navigating to it there is no link for Cookie consents. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Another rebase.. another bug they really aught to have gone in alongside the original cookie consent stuff. Created attachment 165632 [details] [review] Bug 35348: Add cookies link to opac footer This patch adds a link that opens the cookies info dialogue to the opac footer when the CookieConsent feature is enabled. Test plan 1) Enable CookieConsent 2) On the OPAC you should now see a 'Cookies' link at the bottom of the page. 3) Clicking the link should open the "Cookie consent" modal where you can see the details of the cookies you have accepted and change your choices. 4) Bonus points: Enable 'OpacProblemReports' and note it aligns left of 'Cookies' in the same footer bar. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Created attachment 165633 [details] [review] Bug 35348: Ensure users can access cookie details in staff client We remove the check on JSConsents content so that when cookie consent is enabled we retain the option for users to get more information about what cookies are being used. Test plan 1) Enable CookieConsents 2) Do not add any tracking cookies for the staff client into CookieConsentedJS preference 3) Note that 'My consents' appears in the user menu 4) Clicking 'My consents' opens the 'Cookie consent' modal displaying information about the cookies used. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Created attachment 165634 [details] [review] Bug 35348: Remove cookie consents link from opac consents The cookies details modal is now visible for all users from a footer link, we no longer need it under the 'Consents' tab for logged in users. Test plan 1) Enable CookieConsent and add a placholder tracker cookie to CookieConsentedJS preference 2) Log into the OPAC 3) Note that the 'Consents' link no longer appears in the users page. 4) Enable 'PrivacyPolicyConsent' 5) Note that the 'Consents' link now appears but on navigating to it there is no link for Cookie consents. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Not sure about major btw? Passing QA Pushed for 24.05! Well done everyone, thank you! Depends on Bug 35346 not in 23.11.x |