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

Replace feedback form with Smart Survey #8100

Merged
merged 3 commits into from
Jan 30, 2025
Merged

Conversation

jrmhaig
Copy link
Contributor

@jrmhaig jrmhaig commented Jan 27, 2025

To be deployed on 30/1/2025

What

Use Smart Survey for the user feedback form.

Ticket

Migrate Feedback Collection to Smart Survey

Why

The contract with Survey Monkey is expiring and will not be renewed. Smart Survey is the chosen replacement.

How

  • Embed the new survey from Smart Survey as an iframe
  • Delete the Survey Monkey feedback collector form and related code

@jrmhaig jrmhaig force-pushed the CTSKF-1041-smart_survey branch from 9723f61 to bc738c5 Compare January 27, 2025 12:18
@jrmhaig jrmhaig changed the title Replace feedback form with Survey Monkey Replace feedback form with Smart Survey Jan 27, 2025
@jrmhaig jrmhaig force-pushed the CTSKF-1041-smart_survey branch from bc738c5 to c54011b Compare January 27, 2025 12:36
@jrmhaig jrmhaig marked this pull request as ready for review January 27, 2025 13:52
@jrmhaig jrmhaig requested review from a team as code owners January 27, 2025 13:52
Copy link
Contributor

@mpw5 mpw5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@jrmhaig jrmhaig force-pushed the CTSKF-1041-smart_survey branch from c54011b to 76b2611 Compare January 28, 2025 09:24
Copy link

@jrmhaig jrmhaig merged commit f5929c7 into master Jan 30, 2025
18 of 20 checks passed
@jrmhaig jrmhaig deleted the CTSKF-1041-smart_survey branch January 30, 2025 15:25
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.

2 participants