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

[dashboard] Add a few minor missing pages #3760

Merged
merged 3 commits into from
Apr 6, 2021
Merged

Conversation

jankeromnes
Copy link
Contributor

@jankeromnes jankeromnes commented Apr 6, 2021

Missing from https://github.com/gitpod-com/gitpod/issues/5233:

@jankeromnes jankeromnes force-pushed the jx/minor-missing-pages branch 2 times, most recently from 620cb0d to cf754c4 Compare April 6, 2021 12:28
@jankeromnes

This comment has been minimized.

@jankeromnes jankeromnes force-pushed the jx/minor-missing-pages branch 3 times, most recently from bf8ed0c to 8b364ed Compare April 6, 2021 13:20
@jankeromnes jankeromnes marked this pull request as ready for review April 6, 2021 13:22
@jankeromnes
Copy link
Contributor Author

Current state:

Screenshot 2021-04-06 at 15 34 20

Screenshot 2021-04-06 at 15 34 30

Screenshot 2021-04-06 at 15 34 11

@jankeromnes jankeromnes force-pushed the jx/minor-missing-pages branch from 8b364ed to f542176 Compare April 6, 2021 13:37
<p className="mt-4 text-lg text-gitpod-red">{decodeURIComponent(getURLHash())}</p>
</div>
</Route>
<Route path="*" /* status={404} */>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Would be nice to send an actual HTTP 404 status code here, but this requires some server-side hook.

@csweichel
Copy link
Contributor

image

some misalignment on the "blocked" page

@jankeromnes jankeromnes force-pushed the jx/minor-missing-pages branch from f542176 to e53b23a Compare April 6, 2021 13:44
@jankeromnes
Copy link
Contributor Author

jankeromnes commented Apr 6, 2021

some misalignment on the "blocked" page

Thanks. Fixed.

Copy link
Member

@svenefftinge svenefftinge left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@gtsiolis gtsiolis left a comment

Choose a reason for hiding this comment

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

Thanks @jankeromnes for adding instructions how to check these changes!

Looks like a great MVC! 🧱

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.

4 participants