Skip to content
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

fix(UI): Let team admin add built-in/custom repos + delete debug files #56123

Merged
merged 4 commits into from
Sep 13, 2023

Conversation

schew2381
Copy link
Contributor

@schew2381 schew2381 commented Sep 12, 2023

Requires #56148 - backend changes to let team admins delete debug files

In order to check the write access for Team Admins, I need the full project. Originally we were drilling down the project slug instead of the project itself, so I changed it to pass the full project for built-in + custom repo components

Recording of a Team Admin (unable to delete debug file because backend changes are in other PR)

Screen.Recording.2023-09-12.at.6.56.55.PM.mov

Fixes #56116 and #53919

@schew2381 schew2381 self-assigned this Sep 12, 2023
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Sep 12, 2023
@schew2381 schew2381 changed the title fix(UI): Correct team admin not having access to custom repos + debug files fix(UI): Let team admin access built-in repos + custom repos Sep 12, 2023
@schew2381 schew2381 requested a review from a team September 12, 2023 21:56
@schew2381 schew2381 added the Do Not Merge Don't merge label Sep 12, 2023
@schew2381 schew2381 marked this pull request as draft September 12, 2023 23:38
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Sep 13, 2023
@schew2381 schew2381 force-pushed the seiji/fix/let-team-admins-upload-debug-files branch from ad37533 to eee29e5 Compare September 13, 2023 01:47
@getsentry getsentry deleted a comment from github-actions bot Sep 13, 2023
@schew2381 schew2381 removed Do Not Merge Don't merge Scope: Frontend Automatically applied to PRs that change frontend components labels Sep 13, 2023
@schew2381 schew2381 marked this pull request as ready for review September 13, 2023 01:49
@schew2381 schew2381 changed the title fix(UI): Let team admin access built-in repos + custom repos fix(UI): Let team admin access built-in/custom repos + delete debug files Sep 13, 2023
@schew2381 schew2381 changed the title fix(UI): Let team admin access built-in/custom repos + delete debug files fix(UI): Let team admin add built-in/custom repos + delete debug files Sep 13, 2023
schew2381 added a commit that referenced this pull request Sep 13, 2023
…nt (#56148)

Frontend Changes: #56123

Allow team admins to delete debug files through the endpoint

Fixes #53919
@schew2381 schew2381 merged commit 00774a3 into master Sep 13, 2023
90 checks passed
@schew2381 schew2381 deleted the seiji/fix/let-team-admins-upload-debug-files branch September 13, 2023 18:00
@github-actions github-actions bot locked and limited conversation to collaborators Sep 29, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Team Admin Access to Configure Repositories
2 participants