-
Notifications
You must be signed in to change notification settings - Fork 43
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
PIMS-1934 Enhanced Referral Dates #2609
Conversation
🚀 Deployment Information The React APP Image has been built with the tag: |
react-app/src/components/projects/ProjectNotificationsTable.tsx
Outdated
Show resolved
Hide resolved
Much better with the lookup changes there. My only other thought is possibly hiding these values text boxes when the notifications have not been sent out. |
Made this change. If any of the notifications aren't found, it doesn't render the group of date fields. |
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.
Latest change is good. Just gonna merge this cause I want some of these changes in another PR.
🎯 Summary
PIMS-1934
Changes
Testing
🔰 Checklist