generated from unity-sds/unity-repo-template
-
Notifications
You must be signed in to change notification settings - Fork 0
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
New Features for Unity UI #41
Merged
Merged
Conversation
This file contains 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
…a cookie that stores the user token, username, and email of the logged in user. Removed the AuthenticationWrapper that was previously providing login functionality that is no longer needed. Replaced that with an Authorization Wrapper and accompanying auth utils file that provides utility functions for accessing cookies. Updated variable references for token and username throughout application.
… warnings being reported by linting rules.
…nd proxy servers.
…dded new configuration setting to resolve cookie domain issue.
…g updated health json data from the forthcoming introduction of the health api endpoint. Also, resolved issues with the loading of the health json file and the way routes and page titles are handled because of the updated information in the the health json data file.
…rmation to account for proxy paths.
…ger needed. They were being used previously by the AuthenticationWrapper when we were integrating directly with Cognito, which we aren't anymore.
…as they are no longer needed. Also removed unneeded alias configuration in vite.config.ts that referenced these packages. These packages were used by our former AuthenticationWrapper when we integrated directly with Cognito.
…oard. Also renamed and moved CSS class needed by Health Dashboard to _Grid.scss file and updated references to CSS class.
…en't being used at the moment.
…health status information.
…e of health API endpoint. Also fixed how cookies are read.
…PI instance on API GW. Also made improvements to how cell rendering in the health dashboard is handled. And fixed how services in the health dashboard are linked.
…endering correctly in instance of AGGrid.
…n't presently have access to that information from our AWS SSM parameter store.
… issues in AgGrid.
…n errors with Health API Endpoint.
…e for health api endpoint.
…be clicked to navigate to the menu item.
…t unavailability.
rtapella
approved these changes
Nov 11, 2024
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.
Proposed Changes