feat: show error popup when continuing without a course #65

Merged
annadavi merged 2 commits from course-required-popup into ui_change 2026-10-26 03:02:45 +00:00
Member

Description

When a user tried to press continue on account creation without adding a course, it doesn't let them continue but no instructions are shown. Now, a popup appears telling them to add a course in order to continue to the site.

Closes #18

Type of change

New feature

Testing

  • I added/updated tests.
  • I tested my changes locally.
    • I tested my changes with a laptop-size screen.
    • I tested my changes with a phone-size screen.
    • I tested my changes with a tablet-size screen.
  • I tested on the kennel preview deployment for this pull request.

Screenshots/Recordings

Checklist

  • My code follows the project's existing style.
  • I have reviewed my changes.
  • Only the necessary files and lines have been changed.
  • This PR is ready for review.
## Description When a user tried to press continue on account creation without adding a course, it doesn't let them continue but no instructions are shown. Now, a popup appears telling them to add a course in order to continue to the site. ## Related Issue Closes #18 ## Type of change New feature ## Testing - [x] I added/updated tests. - [x] I tested my changes locally. - [x] I tested my changes with a laptop-size screen. - [x] I tested my changes with a phone-size screen. - [x] I tested my changes with a tablet-size screen. - [ ] I tested on the kennel preview deployment for this pull request. ## Screenshots/Recordings ## Checklist - [x] My code follows the project's existing style. - [x] I have reviewed my changes. - [x] Only the necessary files and lines have been changed. - [x] This PR is ready for review.
feat: show error popup when continuing without a course
All checks were successful
kennel/build skipped: no services or static sites defined
Test / Unit tests (push) Successful in 1m24s
Test / Integration tests (push) Successful in 1m33s
Test / Deploy to Production (push) Has been skipped
Test / Deploy to Dev (push) Successful in 1m48s
108b36d20d
annadavi approved these changes 2026-10-04 03:05:20 +00:00
annadavi left a comment

Be sure to update your branch with merge/rebase (using the dropdown below on the right) before creating a merge commit into ui_change!

Be sure to update your branch with merge/rebase (using the dropdown below on the right) before creating a merge commit into ui_change!
sudipi scheduled this pull request to auto merge when all checks succeed 2026-10-24 20:59:39 +00:00
fix: merge ui_change
All checks were successful
nixbot/nix-eval evaluation succeeded in 0s (1 warning)
nixbot/nix-build 0 attributes built
Test / Integration tests (pull_request) Successful in 2m12s
kennel/build build succeeded
kennel/deploy deployment healthy
Test / Unit tests (pull_request) Successful in 1m19s
ab466633d4

Kennel Deployments

Torn down at 2026-10-26 03:02:46 UTC.

<!-- kennel-deployment --> ### Kennel Deployments Torn down at 2026-10-26 03:02:46 UTC.
annadavi canceled auto merging this pull request when all checks succeed 2026-10-26 03:02:40 +00:00
annadavi merged commit ab466633d4 into ui_change 2026-10-26 03:02:45 +00:00
annadavi deleted branch course-required-popup 2026-10-26 03:02:46 +00:00
Sign in to join this conversation.
No description provided.