-
Notifications
You must be signed in to change notification settings - Fork 1.3k
[dashboard] fixing redirects #16297
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
[dashboard] fixing redirects #16297
Conversation
started the job as gitpod-build-se-redirects.1 because the annotations in the pull request description changed |
/werft run with-clean-slate-deployment 👍 started the job as gitpod-build-se-redirects.2 |
f90444a
to
de3c292
Compare
/werft run with-gce-vm=true recreate-preview=true with-preview=true 👍 started the job as gitpod-build-se-redirects.4 |
/werft run with-clean-slate-deployment 👍 started the job as gitpod-build-se-redirects.5 |
de3c292
to
a0fea7d
Compare
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.
A couple small comments, and one bug to check out w/ not being able to change orgs from a project page.
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.
UX LGTM.
Holding for addressing requested changed.
/hold
a0fea7d
to
912b0fd
Compare
I have addressed all feedback |
LGTM. Taking a bias for action and merging this—we can open follow-up issues or PRs for any nitpicks. /unhold |
@selfcontained Seems like you will have to approve this to get this merged. 🏓 |
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.
Change lgtm - fixes the org switching issue, and checks personal projects as well now.
Description
Redirects to the right project (and org) for project URLs that don't exist.
Fixes redirect fro /tokens and /tokens/create
Related Issue(s)
Fixes #16243
How to test
Release Notes
Documentation
Build Options:
Experimental feature to run the build with GitHub Actions (and not in Werft).
leeway-target=components:all
Run Leeway with
--dont-test
Publish Options
Installer Options
Add desired feature flags to the end of the line above, space separated
Preview Environment Options:
If enabled this will build
install/preview
If enabled this will create the environment on GCE infra
Valid options are
all
,workspace
,webapp
,ide
,jetbrains
,vscode
,ssh