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

feat(web): Add micro survey component #6712

Conversation

tonypls
Copy link
Collaborator

@tonypls tonypls commented May 2, 2024

Create a micro survey 📝

Issue

We want to under our users better and how we can improve the app further

Description

This PR uses the feedback card to run a micro survey on how satisfied the users are we the app and prompt them with a dynamic question based on how they review the app.

This is a V1 that points to a Google sheet via Zapier. Eventually we should create an app backend endpoint to handle this for larger volumes and to store both the country and zone which a user is calling the app from.

Testing instructions

Add the ff query param to turn on the micro survey feature flag
Test the survey
Delete the local storage value for "hasSeenSurveyCard"

test(web): don't check for no france

test(web): see if it works with mockserver

install cors

set working dir

ci(web): run mockserver for components test

ci(web): change order

check for mockserver in ci

fix using the fixtures

try with timeout

wait for apis in tests

move wait below

give up on testing jp and stay in europe

don't test for feature colour during load

use first dt

use right click instead of mouseover

split testing into two

only check for exchange arrows
@tonypls tonypls force-pushed the tonyvanswet/avo-193-in-app-micro-survey-app-satisfaction-rating-and-thoughts-on branch from 019a545 to 951e7f2 Compare May 28, 2024 12:35
@tonypls
Copy link
Collaborator Author

tonypls commented May 29, 2024

Ready for review again @madsnedergaard <3

Copy link
Member

@madsnedergaard madsnedergaard left a comment

Choose a reason for hiding this comment

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

LGTM! 🍱

@tonypls tonypls enabled auto-merge (squash) May 29, 2024 12:24
@tonypls tonypls merged commit be2bc43 into master May 29, 2024
21 checks passed
@tonypls tonypls deleted the tonyvanswet/avo-193-in-app-micro-survey-app-satisfaction-rating-and-thoughts-on branch May 29, 2024 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants