-
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
TechDebt: Moment -> DayJs #1189
Conversation
Openshift URLs for the PR Deployment: |
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.
Just navigated through the deployed app a bit, and messed with a few things (edited a survey and funding source, etc). Everything seems happy.
Looks like dayjs is basically a drop-in replacement for moment?
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 failedFailed conditions 7.5% Duplication on New Code (required ≤ 3%) |
Links to Jira Tickets
Description of Changes