Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

allow site admins to list org members & view user/org settings on dotcom#63963

Merged
sqs merged 1 commit into
mainfrom
sqs/site-admin-dc-orgs
Jul 20, 2024
Merged

allow site admins to list org members & view user/org settings on dotcom#63963
sqs merged 1 commit into
mainfrom
sqs/site-admin-dc-orgs

Conversation

@sqs

@sqs sqs commented Jul 19, 2024

Copy link
Copy Markdown
Member

Followup from https://github.com/sourcegraph/sourcegraph/pull/63941.

I forgot to make it so site admins could list org members in that PR.

As for the new ability for site admins to see user and org settings: Site admins could already add themselves to orgs as members and see settings that way. I considered still keeping it stricter, but it is valuable for site admins to be able to view settings to help users troubleshoot.

Test plan

In dotcom mode, as a site admin, view a user or org (that the site admin is not a member of). Confirm that the settings can be viewed.

@sqs sqs requested a review from a team July 19, 2024 23:32
@cla-bot cla-bot Bot added the cla-signed label Jul 19, 2024
@sqs sqs requested a review from a team July 19, 2024 23:34
Followup from https://github.com/sourcegraph/sourcegraph/pull/63941.

I forgot to make it so site admins could list org members in that PR.

As for the new ability for site admins to see user and org settings: Site admins could already add themselves to orgs as members and see settings that way. I considered still keeping it stricter, but it is valuable for site admins to be able to view settings to help users troubleshoot.
@sqs sqs force-pushed the sqs/site-admin-dc-orgs branch from a423026 to 332242f Compare July 20, 2024 00:05

@mohammadualam mohammadualam left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@sqs sqs merged commit 341bfe7 into main Jul 20, 2024
@sqs sqs deleted the sqs/site-admin-dc-orgs branch July 20, 2024 04:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants