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 ID WebCard Cutoff #1250

Closed
1 task done
c3bryant opened this issue Feb 19, 2021 · 4 comments
Closed
1 task done

Staff ID WebCard Cutoff #1250

c3bryant opened this issue Feb 19, 2021 · 4 comments
Assignees
Labels
Bug Something isn't working

Comments

@c3bryant
Copy link
Contributor

c3bryant commented Feb 19, 2021

Time Tracking: MA-34

It was reported that the Staff ID card is rendering in a collapsed state, preventing the user from seeing their full Staff ID card.
Investigate and resolve.

This is done when:

  • Staff ID card handles the case of empty string/null data elements returned from the MyStaffProfile API

@c3bryant c3bryant added the Bug Something isn't working label Feb 19, 2021
@c3bryant c3bryant mentioned this issue Feb 19, 2021
4 tasks
@c3bryant
Copy link
Contributor Author

  • Unable to reproduce with p8gonzal on 2021-02-19
  • Screenshots indicate data elements from MyStaffProfile API like Name, Dept, Emplid were not being returned on 2021-02-18
  • Updating issue with instruction to handle the case where the MyStaffProfile API returns empty or invalid response with status code 200

@c3bryant
Copy link
Contributor Author

Debugged a bit further. When the issue was reported, the API was in a broken state, where it was returning status code 200 (Success) but all data fields were empty strings "", which looked like this:
Screen Shot 2021-02-25 at 11.45.44 AM.png

Resolved a bug where the card height was only being set after rendering the barcode, which was only called if the barcode was not an empty string. If the API enters this broken state again, the card will now look like this:
Screen Shot 2021-02-25 at 11.47.20 AM.png

Verified and deployed to QA & PROD

@woznaut Should we revisit which data elements are required in order to render the Employee ID card metadata vs. falling back to 'An error occurred, try again' user-facing message?

Currently the Staff ID card tries to render any data elements returned (Name, Dept, Emplid, Barcode) and will only show a user-facing error when the API returns a status code that is not 200

@woznaut
Copy link

woznaut commented Mar 8, 2021

Name, EmpID and Barcode should be required. Dept I'm on the fence about.

Error message:
Unable to display Staff ID.
If the problem persists, contact the ITS Service Desk.

@c3bryant
Copy link
Contributor Author

c3bryant commented Mar 9, 2021

Thanks woznaut
I have added issue #1277 for employee id validation

@c3bryant c3bryant closed this as completed Mar 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
Archived in project
Development

No branches or pull requests

2 participants