-
Notifications
You must be signed in to change notification settings - Fork 3
Release to val #1625
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
Merged
Merged
Release to val #1625
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* fix(ui): language for chip eligibility table * removed * * update faq table
🎉 Snyk checks have passed. No issues have been found so far.✅ security/snyk check is complete. No issues have been found. (View Details) ✅ license/snyk check is complete. No issues have been found. (View Details) ✅ code/snyk check is complete. No issues have been found. (View Details) |
* setting up temp feature flag (without actual launchdarkly) * AC 3&4 * add another user role button to my profile * added group & division * restyling ❇️ * add state abbreviation to state card * change state access card header to show role name and add clock icon for pending * fix spelling * I am making a really annoying change from State Acess > Role Status bc I feel like the name made it confusing for cms user profiles :| - sorry * add check helper for state roles and fix abbr for profile cards and fix tests * revert and add test for rolestatuscard feature flag * fix isCMSWithManyRoles for state users * test banner in self revoke * lint * I broke up the status card into multi parts because it is actually kinda complicated?? also added a new card to encapsilate the figma change * set flag to default false also set up LD * hide icons on old UI and update the flag to actually match flag name * fixed broken test & fixed lc connection * remove console log - update feature flag name in the const * convert the components to be conditional based on feature flag instead of passing down the flag * removed extra prop * add requested changes * simplifying the orderedRoleStatus code * updated territory on tests * QA changes and updated lambda * display group/division * proper check for current role * read only always last; approver has read only; button logic updated * added better checks in profile so we dont have failing tests ❇️ * added pre approved to system admin card * removed word user from cms read only role * hiding group and div card on /user and showing the text under myinfo instead * hide pencil when its not their prof * using the userDetails to get group and division instead --------- Co-authored-by: tiffanyvu <[email protected]>
andieswift
approved these changes
Jul 3, 2025
🎉 This PR is included in version 1.5.0-val.196 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Release to val.
Please create a merge commit