Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Commit1d9ad8a

Browse files
committed
Remove org permission-based redirects
I think this needs a proper design/decision.
1 parentebd6dcd commit1d9ad8a

File tree

1 file changed

+1
-22
lines changed

1 file changed

+1
-22
lines changed

‎site/src/pages/ManagementSettingsPage/OrganizationSettingsPage.tsx

Lines changed: 1 addition & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ import type { Organization } from "api/typesGenerated";
1010
import{EmptyState}from"components/EmptyState/EmptyState";
1111
import{displaySuccess}from"components/GlobalSnackbar/utils";
1212
import{Loader}from"components/Loader/Loader";
13-
import{linkToAuditing,withFilter}from"modules/navigation";
1413
import{useOrganizationSettings}from"./ManagementSettingsLayout";
1514
import{OrganizationSettingsPageView}from"./OrganizationSettingsPageView";
1615

@@ -61,28 +60,8 @@ const OrganizationSettingsPage: FC = () => {
6160
}
6261

6362
// When someone views the top-level org URL (/organizations/my-org) they might
64-
// not have edit permissions. Redirect to a page they can view.
65-
// TODO: Instead of redirecting, maybe there should be some summary page for
66-
// the organization that anyone who belongs to the org can read (with
67-
// the description, icon, etc). Or we could show the form that normally
68-
// shows on this page but disable the fields, although that could be
69-
// confusing?
63+
// not have edit permissions.
7064
if(!permissions.editOrganization){
71-
if(permissions.viewMembers){
72-
return<Navigateto="members"replace/>;
73-
}elseif(permissions.viewGrousp){
74-
return<Navigateto="groups"replace/>;
75-
}elseif(permissions.auditOrganization){
76-
return(
77-
<Navigate
78-
to={`/deployment${withFilter(
79-
linkToAuditing,
80-
`organization:${organization.name}`,
81-
)}`}
82-
replace
83-
/>
84-
);
85-
}
8665
return(
8766
<EmptyStatemessage="You do not have permission to edit this organization."/>
8867
);

0 commit comments

Comments
 (0)

[8]ページ先頭

©2009-2025 Movatter.jp