chaynHQ / bloom-frontend

Code for the for the frontend of the Bloom service.
https://bloom.chayn.co/
MIT License
28 stars 46 forks source link

Added ui elements and feedback posting logic #1058

Closed ckirby19 closed 2 months ago

ckirby19 commented 2 months ago

Issue link / number:

858

What changes did you make?

Added session feedback logic and e2e tests to cover what is and isn't shown to the user at each stage The UI can be shown below: HowWasThisSession

After submitting: ThanksForSubmitting

Why did you make the changes?

As requested by the ticket, to enable user feedback on courses

Did you run tests?

Yes, and added additional tests to address the logic of session feedback form Also submitted the form to check that it correctly entered data into the SQL DB

vercel[bot] commented 2 months ago

@ckirby19 is attempting to deploy a commit to the Chayn Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] commented 2 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
bloom-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 23, 2024 9:37am
eleanorreem commented 2 months ago

@ckirby19 great job! 😄 I like how you added the SessionFeedback at Page level. Only a couple of comments to get it over the line. Thanks again!

eleanorreem commented 2 months ago

I forgot to add this to the ticket but there is another template to add this to /bloom-frontend/pages/courses/image-based-abuse-and-rebuilding-ourselves/[sessionSlug].tsx. It's only used on the Image based abuse course. I will create a new ticket for this as this was my omission

cypress[bot] commented 2 months ago

Passing run #616 ↗︎

0 40 2 0 Flakiness 0
⚠️ You've recorded test results over your free plan limit.
Upgrade your plan to view test results.

Details:

feat: Session Feedback (#1058)
Project: Bloom frontend Commit: 32c4de37e3
Status: Passed Duration: 06:23 💡
Started: Jul 23, 2024 10:56 AM Ended: Jul 23, 2024 11:02 AM

Review all test suite changes for PR #1058 ↗︎