-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
chore(ui): move users & about nav items #17849
Conversation
e08860b
to
268d31c
Compare
b431bfe
to
d85947b
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.
Make sure to update the router as well
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.
Curious: is the quartz issue also being done? The issues really emphasize them being done together.
CHANGELOG.md
Outdated
1. [17849](https://github.com/influxdata/influxdb/pull/17849): Move Organization navigation items to user menu. | ||
1. [17714](https://github.com/influxdata/influxdb/pull/17714): Cloud environments no longer render markdown images, for security reasons. | ||
1. [17321](https://github.com/influxdata/influxdb/pull/17321): Improve UI for sorting resources | ||
1. [17740](https://github.com/influxdata/influxdb/pull/17740): Add single-color color schemes for visualizations |
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.
nit: i think we append to the bottom not the top. not that it's a very big deal
1. [17849](https://github.com/influxdata/influxdb/pull/17849): Move Organization navigation items to user menu. | |
1. [17714](https://github.com/influxdata/influxdb/pull/17714): Cloud environments no longer render markdown images, for security reasons. | |
1. [17321](https://github.com/influxdata/influxdb/pull/17321): Improve UI for sorting resources | |
1. [17740](https://github.com/influxdata/influxdb/pull/17740): Add single-color color schemes for visualizations | |
1. [17714](https://github.com/influxdata/influxdb/pull/17714): Cloud environments no longer render markdown images, for security reasons. | |
1. [17321](https://github.com/influxdata/influxdb/pull/17321): Improve UI for sorting resources | |
1. [17740](https://github.com/influxdata/influxdb/pull/17740): Add single-color color schemes for visualizations | |
1. [17849](https://github.com/influxdata/influxdb/pull/17849): Move Organization navigation items to user menu. |
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.
Ya, here's the Quartz PR: https://github.com/influxdata/quartz/pull/2530
linkElement={className => ( | ||
<a | ||
className={className} | ||
href={`${CLOUD_URL}/organizations/${org.id}${CLOUD_USERS_PATH}`} |
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.
what's up with /orgs/
and /organizations/
?
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.
This is the route we ended up within Quartz, there's discussions ongoing around syncing this and users/members but that's outside the scope of this PR.
@mavarius can you throw some screenshots of the changes in here? |
Might want to rename |
Actually, disregard my last comment |
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.
👍 But let's coordinate this deploy with cloud2.influxdata.com
No changes need to be made to the routes, only the UI elements. "Members" is only renamed to "Users" in cloud, not OSS. |
e665249
to
5028b8a
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.
Groovy
Closes #17578