Features/unity sds client update health service #43
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
…PI that has been migrated to AWS API GW. These changes include an updated health json schema, updated API calls in the health service, and updated printing of the health status information.
|
Interesting, three of the jobs failed when the PR was created, no changes made to the PR, reran the three failed jobs and they all passed. 😕 |
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.
Purpose
This PR addresses the following changes:
Proposed Changes
landingPageUrlinformation is included in the health status report.Issues
unity-sds/unity-py#91
@rtapella I can't comment or transfer the above ticket to the monorepo repository because the unity-py repo has been archvied.
Testing
Successfully ran unit tests to confirm above changes are working as expected against our Unity-Venue-Dev account