Skip to content

Conversation

@billyvg
Copy link
Member

@billyvg billyvg commented May 1, 2025

This should fix this issue as the /settings/ index page can link to invalid URLs when user is not apart of any orgs. (I actually think this only is visitable when the user is not part of any orgs, otherwise it redirects you to org settings). Replace/remove all org links on this index page.

Also fixes text clipping due to overflow + ellipsis (look at the "g" in Organization)

Old:
image

New:

image

Fixes JAVASCRIPT-2M6X

This should fix [todo] as the `/settings/` index page can link to invalid URLs when user is not apart of any orgs. Replace/remove all org links on this index page. Also fixes text clipping due to overflow + ellipsis.
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 1, 2025
@billyvg billyvg changed the title feat(ui): Fix /settings/ when no org in context feat(ui): Fix /settings/ when user is not part of any orgs May 1, 2025

const SelfHostedLink = icon ? HomeLinkIcon : HomeLink;
return <SelfHostedLink to={`${organizationSettingsUrl}support`} {...props} />;
return <SelfHostedLink to="https://sentry.zendesk.com/hc/en-us" {...props} />;
Copy link
Member Author

Choose a reason for hiding this comment

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

The prev support link, /settings/:orgSlug/support/ redirects to this.

@billyvg billyvg marked this pull request as ready for review May 1, 2025 21:21
@billyvg billyvg requested a review from a team May 1, 2025 21:21
@billyvg billyvg merged commit 065fe65 into master May 1, 2025
41 checks passed
@billyvg billyvg deleted the feat-ui-fix-org-links-in-settings branch May 1, 2025 21:38
andrewshie-sentry pushed a commit that referenced this pull request May 12, 2025
This should fix [this
issue](https://sentry.sentry.io/issues/4191022159/?project=11276&query=is%3Aunresolved%20issue.priority%3A%5Bhigh%2C%20medium%5D%20useOrganization&referrer=issue-stream&stream_index=1)
as the `/settings/` index page can link to invalid URLs when user is not
apart of any orgs. (I actually think this *only* is visitable when the
user is not part of any orgs, otherwise it redirects you to org
settings). Replace/remove all org links on this index page.

Also fixes text clipping due to overflow + ellipsis (look at the "g" in
Organization)


Old:

![image](https://github.com/user-attachments/assets/83c0013b-4a91-40eb-bdbc-05473ac7127a)


New:


![image](https://github.com/user-attachments/assets/2d246689-8735-467f-9461-6bdb48c9f9bd)

Fixes JAVASCRIPT-2M6X
@github-actions github-actions bot locked and limited conversation to collaborators May 17, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants