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

Commit01c4b19

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

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

@@ -59,28 +58,8 @@ const OrganizationSettingsPage: FC = () => {
5958
}
6059

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

0 commit comments

Comments
 (0)

[8]ページ先頭

©2009-2025 Movatter.jp