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

chor: change cluster name param to have datacenter.name as default value #20644

Conversation

valeriia-ruban
Copy link
Contributor

Description

rethink on default value for cluster name on linking.
With the pr it is set to datacenter name

Attention to link ⬇️
image

Testing & Reproduction steps

  1. fo to consul
  2. check the link to hcp modal and click/hover the Next button
  3. see the cluster_name param in the link bellow/in a new tab when redirected

Links

PR Checklist

  • updated test coverage
  • external facing docs updated
  • appropriate backport labels added
  • not a security concern

@github-actions github-actions bot added the theme/ui Anything related to the UI label Feb 14, 2024
@valeriia-ruban valeriia-ruban added backport/1.18 pr/no-changelog PR does not need a corresponding .changelog entry pr/no-metrics-test labels Feb 14, 2024
@valeriia-ruban valeriia-ruban merged commit 8425cd0 into main Feb 15, 2024
96 checks passed
@valeriia-ruban valeriia-ruban deleted the change-from-api-returned-cluster-name-to-data-center-name-on-link-to-hcp branch February 15, 2024 09:26
chris-hut added a commit that referenced this pull request Mar 11, 2024
chris-hut added a commit that referenced this pull request Mar 13, 2024
* Revert "feat: add alert to link to hcp modal to ask a user refresh a page; up… (#20682)"

This reverts commit dd833d9.

* Revert "chor: change cluster name param to have datacenter.name as default value (#20644)"

This reverts commit 8425cd0.

* Revert "chor: adds informative error message when acls disabled and read-only… (#20600)"

This reverts commit 9d712cc.

* Revert "Cc 7147 link to hcp modal (#20474)"

This reverts commit 8c05e57.

* Revert "Add nav bar item to show HCP link status and encourage folks to link (#20370)"

This reverts commit 22e6ce0.

* Revert "Cc 7145 hcp link status api (#20330)"

This reverts commit 049ca10.

* Revert "💜 Cc 7187/purple banner for linking existing clusters (#20275)"

This reverts commit 5119667.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/no-changelog PR does not need a corresponding .changelog entry pr/no-metrics-test theme/ui Anything related to the UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants