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

NET-5398: V2 unavailable UI message #20359

Merged
merged 7 commits into from
Jan 29, 2024
Merged

Conversation

WenInCode
Copy link
Contributor

@WenInCode WenInCode commented Jan 26, 2024

Description

This is currently based off #20353 until that gets merged

Adds a redirect to a /unavailable page that shows a message about the UI being unvailable if you have v2 experiment enabled. It also redirects away from the unavailable page if it is not enabled.

Screenshot 2024-01-26 at 9 42 18 AM Screenshot 2024-01-26 at 9 40 44 AM

Testing & Reproduction steps

I mainly tested this by building Consul and enabling the experiment in the server config and then opening the UI (steps in this ticket).

You can do the naïve test as well by running this against the mocks and then running Scenario('CONSUL_V2_CATALOG_ENABLE=1') in the console.

Links

#20353
https://hashicorp.atlassian.net/browse/NET-5398
https://hashicorp.atlassian.net/browse/CC-6956

PR Checklist

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

@WenInCode WenInCode added theme/ui Anything related to the UI pr/no-backport labels Jan 26, 2024
@WenInCode WenInCode force-pushed the NET-5398/v2-unavailable-ui branch from c08a36a to f5a848c Compare January 26, 2024 17:32
{{#if (env 'CONSUL_V2_CATALOG_ENABLED')}}
<div class='h-screen w-full flex flex-col justify-center items-center'>
<div>
<svg
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 just used an SVG here from the figma because this is a logo with the text, but the text is using our marketing fonts (?).

@WenInCode WenInCode marked this pull request as ready for review January 26, 2024 18:25
@WenInCode WenInCode force-pushed the NET-5398/add-V2Enabled-to-config branch from 32a25b8 to 2b8feba Compare January 26, 2024 19:50
@WenInCode WenInCode force-pushed the NET-5398/v2-unavailable-ui branch from 3f9d066 to 56ddeea Compare January 26, 2024 19:52
Base automatically changed from NET-5398/add-V2Enabled-to-config to main January 26, 2024 21:38
Copy link
Contributor

@chris-hut chris-hut left a comment

Choose a reason for hiding this comment

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

Nice!
Just a couple questions about doubling down

.changelog/20353.txt Outdated Show resolved Hide resolved
ui/packages/consul-ui/app/routes/application.js Outdated Show resolved Hide resolved
ui/packages/consul-ui/app/templates/unavailable.hbs Outdated Show resolved Hide resolved
@WenInCode WenInCode merged commit b9f3e5e into main Jan 29, 2024
100 of 103 checks passed
@WenInCode WenInCode deleted the NET-5398/v2-unavailable-ui branch January 29, 2024 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/no-backport theme/ui Anything related to the UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants