Merge cleanup #1583
Merged
Merge cleanup #1583
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
🔧 Hotfix registration
* Verify that user in tenant (#1563) Co-authored-by: davidreneuw <[email protected]> * ✨ New home page (#1556) * Mock home page and drop down in sidebar * Home page mocks * New type of card * New home page * Improve workspace sidebar dropdown * Adding localization and sorting * Only show non-deleted workspaces in dropdown * Make cost bar horizontal, add label on top * Remove debug leftover --------- Co-authored-by: datahub-portal-pr-rw[bot] <192352896+datahub-portal-pr-rw[bot]@users.noreply.github.com> Co-authored-by: Erik <[email protected]> * Handle missing credits better (#1570) * 💰 Updating the GC Hosting endpoint fields (#1568) * Adding migration to change what GC Hosting gives us * Updating controller to handle new fields * Updating tool page to handle new fields * Updating tool page to add the workspace budget (missed before --------- Co-authored-by: datahub-portal-pr-rw[bot] <192352896+datahub-portal-pr-rw[bot]@users.noreply.github.com> * Postgres Tier code refactor (#1564) * Postgres tiers merging * code cleanup * Restricting usage emails to admins and workspace leads (#1565) * Fixing styling issue with HomeNav (#1559) * 🐛 Parametrize budget rollover into a configuration (#1562) * Parametrize budget rollover into a configuration * Fix logic to give the force option the highest authority * Adjust for unit tests --------- Co-authored-by: datahub-portal-pr-rw[bot] <192352896+datahub-portal-pr-rw[bot]@users.noreply.github.com> * Adding migration for new fields (#1573) * Added unit tests for GC Hosting API (#1571) * - misc validation for hosting service controller - removed WorkspaceIdentifier since acronym is returned * added request id * added feature for gc hosting * Add files via upload * moved test files * fixed workspace storage tests * added sample requests and unit tests for hosting services api * added valid sample for latest format * fix for unit tests * fixed required values in json --------- Co-authored-by: datahub-portal-pr-rw[bot] <192352896+datahub-portal-pr-rw[bot]@users.noreply.github.com> --------- Co-authored-by: davidreneuw <[email protected]> Co-authored-by: David René <[email protected]> Co-authored-by: datahub-portal-pr-rw[bot] <192352896+datahub-portal-pr-rw[bot]@users.noreply.github.com> Co-authored-by: Sean Stilwell <[email protected]> Co-authored-by: gcnabeel <[email protected]>
additional validation for fields
ErikApption
requested review from
Sean-Stilwell,
davidreneuw and
gcnabeel
as code owners
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.
Pull Request
Description
Checklist