-
Notifications
You must be signed in to change notification settings - Fork 9
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
SIMSBIOHUB-620: Manage System Alerts #1401
Conversation
…funding sources patterns
Openshift URLs for the PR Deployment: |
Openshift URLs for the PR Deployment: |
Converting to draft temporarily to save Openshift resources. |
app/src/features/admin/users/access-requests/list/pending/AccessRequestPendingList.tsx
Show resolved
Hide resolved
@mauberti-bc Really slick in general, nice work! I like the way the alerts are displayed in the admin page and form (with the preview). 2 things I noticed when adding/editing/deleting alerts in the UI:
Regarding CSS: |
Openshift URLs for the PR Deployment: |
Fixed the bug preventing end dates from being removed by updating the DateField component to correctly catch invalid dates. Also changed the max length of alert name to 50 in the database. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quality Gate passedIssues Measures |
Links to Jira Tickets
Description of Changes
Testing Notes