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.

Sign up for GitHub

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

Jump to bottom

Issue/hitide UI 68 - fix dataset ui crash bug #69

Merged
merged 4 commits into from
May 20, 2024
Merged

Issue/hitide UI 68 - fix dataset ui crash bug #69

merged 4 commits into from
May 20, 2024

Conversation

jbyrne6
Copy link
Contributor

@jbyrne6 jbyrne6 commented May 20, 2024 •
edited
Loading

Changes

  • Fixed error handling of metadata missing to prevent erroring out

Related

Testing

  • Not working in SIT
  • Working in UAT

TODO

  • Handle database error gracefully so frontend doesn't crash

@jbyrne6 jbyrne6 added the bug Something isn't working label May 20, 2024
@jbyrne6 jbyrne6 linked an issue May 20, 2024 that may be closed by this pull request
When adding MERGED_TP_J1_OSTM_OST_CYCLES_V52 to HiTIDE UAT it breaks HiTIDE, update HiTIDE to not fail when this happens but make a warning #68
Closed
@jbyrne6 jbyrne6 self-assigned this May 20, 2024
@jamesfwood jamesfwood merged commit 3606172 into develop May 20, 2024
3 checks passed
@jamesfwood jamesfwood deleted the issue/hitide-ui-68 branch May 20, 2024 23:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jamesfwood jamesfwood jamesfwood approved these changes

@davidcolemanjpl davidcolemanjpl Awaiting requested review from davidcolemanjpl

@sliu008 sliu008 Awaiting requested review from sliu008

Assignees

@jbyrne6 jbyrne6

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
2 participants
@jbyrne6 @jamesfwood