Skip to content
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

Sync UI: Add granularity to sync destinations #25500

Merged
merged 11 commits into from
Feb 20, 2024

Conversation

hellobontempo
Copy link
Contributor

@hellobontempo hellobontempo commented Feb 16, 2024

UI updates adding granularity field to a sync destination.

  • Added granularity field to destination form
  • Added secret key badge to list view
  • Add copy to popup menu so user knows any actions performed on a key will apply to the secret, not the individual key. I updated to use the Hds::Dropdown so that adding copy to the popup menu was cleaner

updates

@hellobontempo hellobontempo added this to the 1.16.0 milestone Feb 16, 2024
@github-actions github-actions bot added the hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed label Feb 16, 2024
Copy link

github-actions bot commented Feb 16, 2024

Build Results:
All builds succeeded! ✅

Copy link

github-actions bot commented Feb 16, 2024

CI Results:
All Go tests succeeded! ✅

Copy link
Contributor

@Monkeychip Monkeychip left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I probably have the least context on the team re: secret sync, so please reach out to Jordan if you feel the extra review is needed. However, I also know ga freeze is soon, so didn't want you blocked.

<span>{{or val.label val.value val}}</span>
{{or val.label val.value val}}
{{#if val.helpText}}
<Hds::TooltipButton @text={{val.helpText}} aria-label="More information">
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I used Hds here because InfoTooltip styling was too low and didn't have the patience to CSS

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Screenshot 2024-02-16 at 8 17 23 PM

@@ -38,6 +38,11 @@
Select a KV engine mount and path to sync a secret to the
{{@destination.typeDisplayName}}
destination. Selecting a previously synced secret will re-sync that secret.
{{#if (eq @destination.granularity "secret-key")}}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Screenshot 2024-02-16 at 8 03 37 PM

<li>
<LinkTo
class="has-text-black has-text-weight-semibold"
<Item.menu @hasMenu={{false}}>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hasMenu is false so that we can use the Hds::Dropdown istead of PopupMenu

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

replaced destinations popup menu so the views were consistent

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Screenshot 2024-02-16 at 8 15 00 PM

@zofskeez zofskeez merged commit dd62f9f into main Feb 20, 2024
31 checks passed
@zofskeez zofskeez deleted the ui/VAULT-24177/add-granularity-to-sync-destinations branch February 20, 2024 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants