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

Staff Check-in Error Handling Fix #601

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

nathannwangg
Copy link

@nathannwangg nathannwangg commented Dec 13, 2024

Description

Fixes #599

  • Fixed bug in error checking for incorrect code "402" instead of the correct code "404"
  • Updated alert error messages to be a little more descriptive

…r alert messages to be a little more descriptive.
@nathannwangg nathannwangg changed the title Staff Check-in Error Checking Fix Staff Check-in Error Handling Fix Dec 13, 2024
@anushkasankaran
Copy link
Contributor

The API should return an error message. Would we be able to display what's returned when there's an error instead of hardcoding the phrases?

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.

Staff Attendance Error Handling
2 participants