-
Notifications
You must be signed in to change notification settings - Fork 172
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
Zarf UI rewrite #1522
Merged
Merged
Zarf UI rewrite #1522
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
## Description #1500 ## Related Issue #1500 ## Type of change - [x] UI Change --------- Co-authored-by: Jessy-Morris <[email protected]>
✅ Deploy Preview for zarf-docs ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
…1593) Create deployed package menu. Update deployed-package-table to use the new deploy package menu. Removed package-err-not-found component. Create spinner-component. Fix cluster info not disconnecting cluster when it is destroyed. Cleaned up unused imports and vars in nav-drawer. Update unicornui ## Description ... ## Related Issue #1571
mike-winberry
requested review from
Racer159,
Noxsios,
jeff-mccoy,
YrrepNoj and
a team
as code owners
April 12, 2023 14:57
Noxsios
reviewed
Apr 12, 2023
Noxsios
reviewed
Apr 12, 2023
Noxsios
reviewed
Apr 12, 2023
Noxsios
reviewed
Apr 12, 2023
Noxsios
reviewed
Apr 12, 2023
Noxsios
reviewed
Apr 12, 2023
Noxsios
reviewed
Apr 12, 2023
Noxsios
reviewed
Apr 12, 2023
Noxsios
reviewed
Apr 12, 2023
Noxsios
reviewed
Apr 12, 2023
Noxsios
reviewed
Apr 12, 2023
Noxsios
reviewed
Apr 12, 2023
Noxsios
reviewed
Apr 12, 2023
Noxsios
reviewed
Apr 12, 2023
…thorization header. Update zarfui with the fetch-event-source dependency to allow the passing of the authorization header when connecting to event streams. Update ui/lib http to implement the fetch-event-source within the connect method which now returns an AbortController to close and cleanup the stream, renamed deployStream to eventStream that takes in EventParams. Update ui/lib api to reflect changes made to http. Update deployment page to reflect the changes to the event stream logic.
…r to fix the hanging html tag issue. Update package.json with format command to run prettier against all svelte, js, and ts files within the src/ui folder. Update prettierignore with path to the ui api-types.ts as that is generated with the hacks and formatting could break ci comparison.
|
… having to ignore errors. Override palette.ts dark theme disabled and on-disabled to match design. Update button-dense to properly apply disabled background colors when the background color is partially transparent, created issue in UUI to resolve this edge case for the button component. Create zarf dialog component that customizes the uui dialog and allows choosing between sad and happy zarf. Update connect-cluster-dialog to use the newly created zarf-dialog. Update deploy-package menu with the remove verification flow. Fix error in deploy-utils getDeployedComponents looking for index in undefined list.
…text to the happy or sad zarf image. Update deployed-package-menu and connect-cluster-dialog with specific zarfAlt text.
Screen.Recording.2023-04-15.at.1.40.47.AM.mov |
Racer159
requested changes
Apr 17, 2023
Co-authored-by: Wayne Starr <[email protected]>
Co-authored-by: Wayne Starr <[email protected]>
…nit packages that are corrupted or unable to open, Added note on #1312 for error handling while searching packages.
Co-authored-by: Wayne Starr <[email protected]>
Racer159
approved these changes
Apr 18, 2023
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.
lgtm
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.
Description
Fixes: #1500
Fixes: #1311
Closes
Fixes: #964
Fixes: #807
Fixes: #794
Fixes: #1500
Fixes: #1311