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

Feedback Module #91

Open
JonathanDHolden opened this issue Sep 6, 2024 · 1 comment · May be fixed by #92
Open

Feedback Module #91

JonathanDHolden opened this issue Sep 6, 2024 · 1 comment · May be fixed by #92
Assignees

Comments

@JonathanDHolden
Copy link

We're looking for a Feedback module... Something like the module at the bottom of every gov.uk page, for example https://www.gov.uk/service-manual

image

Discusson from Slack https://localgovdrupal.slack.com/archives/C020SJRQX2M/p1723623283830739

Jonny Holden (Calderdale / poppop)
14 Aug at 10:14 AM
Is there a "Give feedback on this page" component_ I see Essex have a form, but I wonder if there is a thing like this from Gov
image.png

image.png

13 replies

Mark Conroy (mark.ie)
14 Aug at 10:46 AM
We haven’t built this yet. Most councils seem to want it, so might be a good candidate for the backlog.
👍
3

Will (LocalGov Drupal)
14 Aug at 11:13 AM
Up until now councils have preferred using their own forms/ case management to collect user feedback, so we’ve prioritised other features. It would be good to chat about this here and at LocalGov Csmp

Mark Conroy (mark.ie)
14 Aug at 11:36 AM
A generic version would be pretty simple to make, and if councils wanted to use their own solution instead they could just ignore the simpler generic version (or even build on top of it).
👍
1

Will (LocalGov Drupal)
14 Aug at 4:37 PM
I’d like to roll this into the insight stuff we’ve been talking about. Catch up when I’m back from holiday?
👍
1

ekes (agile/iskra)
14 Aug at 11:29 PM
I assume several of these will use webform, which then has different backends to push to (from the simple email it somewhere, to putting it into CRMs).
By making a recipe or module for it. We could do the configuration such that we know we aren't breaking caching of the page - which is easy enough to do - and also include good default wording (like don't submit personal information with this form etc.)
11:31
Spending some time so it could be loaded off page with a bit of javascript would be nice for example. So cached pages, plus javascript requirement to put off bots.
❤️
1

Lizzi Cox (Bracknell Forest)
20 Aug at 9:47 AM
We had this using a block and a web form. But we just removed ours as the spam filter applied to it was blocking 90% of traffic. We need to look at a new solution that us both useful and prevents the endless spam we got through it. Looking at alternative soltuins now.

Jonny Holden (Calderdale / poppop)
20 Aug at 9:58 AM
👍

Mark Conroy (mark.ie)
20 Aug at 10:10 AM
the spam filter applied to it was blocking 90% of traffic
Can you explain that a little bit more
@Lizzi Cox (Bracknell Forest)
?

Lizzi Cox (Bracknell Forest)
20 Aug at 10:21 AM
@mark Conroy (mark.ie)

  • Not really! 😄 We used something called 'Clean Talk' to filter spam and it was blocking our Fastly CDN IP addresses (this is what I got from our dev team so I may have this wrong).

Mark Conroy (mark.ie)
20 Aug at 10:22 AM
Ah that makes sense. I thought it was something in the Drupal set up that was causing traffic to be blocked.
👍
1

ekes (agile/iskra)
20 Aug at 10:28 AM
Searching 'Clean Talk' Drupal I guess it was using the Drupal Module (with the SaaS offering from CleanTalk - whose GDPR statement doesn't fill me with hope tbh).
That said unless the module was written badly, or Drupal isn't configured to be behind a CDN, it should have been using X-Forwarded for headers (ie the clients IP address, not the CDNs).
10:31
I think (as suggested above) loading the form via ajax, it can even have a unique token then - preventing multiple or direct submissions, will probably solve most spam problems, without requirement for an external service.

@markconroy markconroy transferred this issue from localgovdrupal/localgov Sep 9, 2024
@markconroy markconroy linked a pull request Sep 9, 2024 that will close this issue
@markconroy
Copy link
Member

markconroy commented Sep 9, 2024

Hi @JonathanDHolden

Here's a PR with a feedback form for you if you'd like to test it.

#92


Thanks to Big Blue Door for sponsoring my time to work on this.

@finnlewis finnlewis self-assigned this Dec 10, 2024
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 a pull request may close this issue.

3 participants