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

feat: 🎸 Added GCP host sets and hosts #2658

Closed
wants to merge 7 commits into from

Conversation

ZedLi
Copy link
Collaborator

@ZedLi ZedLi commented Jan 10, 2025

✅ Closes: https://hashicorp.atlassian.net/browse/ICU-15994
Note this is a duplicate of #2655 which got accidentally lost in a force push.

Description

This adds support for GCP host sets and hosts. I also updated the styles for the host page in all 3 dynamic catalogs to add some space to the dynamic inputs. Will add tests.

Screenshots (if appropriate)

Host Set page:
image

Host List page:
image

Host page:
image

AWS host page:
image

AWS before:
image

How to Test

Checklist

  • I have added before and after screenshots for UI changes
  • [ ] I have added JSON response output for API changes
  • [ ] I have added steps to reproduce and test for bug fixes in the description
  • I have commented on my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works

@ZedLi ZedLi self-assigned this Jan 10, 2025
@ZedLi ZedLi requested a review from a team as a code owner January 10, 2025 21:44
Copy link

vercel bot commented Jan 10, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
boundary-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 10, 2025 9:45pm
boundary-ui-desktop ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 10, 2025 9:45pm

cameronperera
cameronperera previously approved these changes Jan 10, 2025
@DhariniJeeva
Copy link
Collaborator

apologies for losing this commit, thanks for recreating this 😿

DhariniJeeva
DhariniJeeva previously approved these changes Jan 10, 2025
@DhariniJeeva DhariniJeeva dismissed stale reviews from cameronperera and themself January 10, 2025 22:18

The merge-base changed after approval.

@ZedLi ZedLi closed this Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants