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: Add finish button and integrate with feedback screen #180

Merged
merged 3 commits into from
Jan 26, 2025

Conversation

frederickbaier
Copy link
Contributor

Added the finish button at the end of the playtime session which redirects to the feedback screen.
The finish button is shown on the last step instead of the 'next' button.
Refactored feedback screen to submit the feedback into the correct activity.
Fixed NewKid flow to reset properly after completion.

…regenerate button and integrate with feedback screen

Signed-off-by: Frederick Baier <[email protected]>
@github-actions github-actions bot added the release: feature New features and functionalities added to the project. label Jan 25, 2025
Copy link

👋 Hi there!

🚀 Checkout Instructions

git fetch origin
git checkout refactor/playtime-session

Happy coding! 🎉

Copy link

github-actions bot commented Jan 25, 2025

🚀 Expo preview is ready!

  • Project → cognikids
  • Platforms → android, ios
  • Scheme → cognikids
  • Runtime Version → exposdk:51.0.0
  • More info

Learn more about 𝝠 Expo Github Action

Copy link
Contributor

@shanaka95 shanaka95 left a comment

Choose a reason for hiding this comment

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

Reviewed and Tested the changes. Fixed some merge conflicts with the latest code in the main branch. Code looks good. Functionalities are working. Ready to merge!

@frederickbaier frederickbaier merged commit 68d91ce into main Jan 26, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release: feature New features and functionalities added to the project.
Projects
Status: PR Archive
Development

Successfully merging this pull request may close these issues.

2 participants