-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
UHF-5047 map visibility cookie consent #540
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Almost. :)
Please see my comments below.
This branch is missing the changed dist folder files. Please create them before merging.
34b268e
to
e1c5c04
Compare
Good catches! All should be fixed now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works now, good job!
UHF-5047
Maps were shown even though cookie consent was not given.
What was done
How to install
git pull origin dev
composer require drupal/hdbt:dev-UHF-5047_Map-visibility-cookie-consent
make drush-cr
How to test
Designers review