Skip to content

fix: Adjust welcome and end screen#68

Merged
esterlus merged 7 commits intomainfrom
este/init
Mar 23, 2026
Merged

fix: Adjust welcome and end screen#68
esterlus merged 7 commits intomainfrom
este/init

Conversation

@esterlus
Copy link
Contributor

Fixes #61
Fixes #67

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the onboarding flow to remove the (currently non-functional) anonymous path and to show a dedicated end-of-onboarding thank-you message after feedback submission, aligning the UI with Issues #61 and #67. It also adds Nix/direnv developer environment files and updates ignore rules.

Changes:

  • Remove the “Stay anonymous” option from the login screen and adjust onboarding privacy copy.
  • Add a “Thank you” dialog shown after submitting the final feedback step instead of immediately returning to login.
  • Add Nix flake + direnv config for a reproducible dev shell; update .gitignore accordingly (and refresh yarn.lock).

Reviewed changes

Copilot reviewed 4 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/views/Login.tsx Removes anonymous path button; updates onboarding/privacy messaging and login CTA.
src/views/onboarding/99_Feedback.tsx Adds thank-you dialog flow after “The End” submission.
flake.nix Introduces a Nix flake-based dev shell for the project.
flake.lock Pins nixpkgs revision for the flake.
.envrc Enables direnv to load the flake dev shell automatically.
.gitignore Ignores .direnv directory.
yarn.lock Updates lockfile with additional/changed transitive dependencies.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.

Add Thank you Screen Temporarily remove "Stay anonymous" path

3 participants