Skip to content
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

Show Google Form feedback modal on Area and Explore pages #576

Merged
merged 1 commit into from
Jul 5, 2024

Conversation

zarino
Copy link
Member

@zarino zarino commented Jul 3, 2024

Screenshot 2024-07-03 at 17-43-50 Explore Local Intelligence Hub

Markup, styling, and JavaScript handling for feedback modal that only gets shown on Area and Explore pages if the user hasn’t already skipped or submitted it. Form is submitted, via Ajax, to a Google Form in the mySociety/TCC shared drive.

As part of this, I reused and refactored the CollapsableMailingListForm from the Area page, into a general setUpCollapsable function.

I also fixed a bug where the "shared" JavaScript in home-out-esm.js (such as the .form-check + .conditional-fields handling) wasn’t being loaded on Area or Explore pages. I guess, really, we should either rename home-out-esm.js to shared-out-esm.js, or we should break out the parts that are intended to be used on other pages, and leave home-out-esm.js for homepage-only stuff.

And since the feedback form gathers some personal information, I’ve updated the Privacy Policy to explain that.

@zarino zarino requested a review from struan July 3, 2024 16:45
Markup, styling, and JavaScript handling for feedback modal that only
gets shown on Area and Explore pages if the user hasn’t already skipped
or submitted it. Form is submitted, via Ajax, to a Google Form.

As part of this, I reused and refactored the CollapsableMailingListForm
from the Area page, into a general setUpCollapsable function.

I also fixed a bug where the "shared" JavaScript in home-out-esm.js
(such as the `.form-check + .conditional-fields` handling) wasn’t being
loaded on Area or Explore pages. I guess, really, we should either
rename home-out-esm.js to shared-out-esm.js, or we should break out
the parts that are intended to be used on other pages, and leave
home-out-esm.js for homepage-only stuff.

And since the feedback form gathers some personal information, I’ve
updated the Privacy Policy to explain that.
@zarino zarino merged commit a433471 into main Jul 5, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant