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

Edit support users in program tab #4156

Merged
merged 11 commits into from
Sep 19, 2024

Conversation

rpiaggio
Copy link
Contributor

@rpiaggio rpiaggio commented Sep 18, 2024

Invitations can now have roles. Only PIs and CoIs are now shown in the Proposal tab. This PR also paves the way to editing support users in the Program tab.

Edit support users in the program tab. Only Admin and Staff users can do this. Support users were removed from the Proposal tab.

image

@rpiaggio rpiaggio force-pushed the sc-3149-move-support-users-to-the-program-tab branch from 0149f47 to db220e0 Compare September 18, 2024 03:18
Copy link

bundlemon bot commented Sep 18, 2024

BundleMon

Files updated (2)
Status Path Size Limits
index-(hash).js
1.69MB (+1.16KB +0.07%) -
index-(hash).css
65.79KB (+56B +0.08%) -
Unchanged files (6)
Status Path Size Limits
exploreworkers-(hash).js
608.79KB -
workbox-window.prod.es5-(hash).js
2.07KB -
plotworker-(hash).js
85B -
agsworker-(hash).js
84B -
itcworker-(hash).js
82B -
catalogworker-(hash).js
81B -

Total files change +1.21KB +0.05%

Final result: ✅

View report in BundleMon website ➡️


Current branch size history | Target branch size history

@rpiaggio rpiaggio changed the title Invitations with roles Edit support users in program tab Sep 19, 2024
Copy link
Contributor

@cquiroz cquiroz left a comment

Choose a reason for hiding this comment

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

LGTM

@rpiaggio rpiaggio merged commit e1d522c into master Sep 19, 2024
14 checks passed
@rpiaggio rpiaggio deleted the sc-3149-move-support-users-to-the-program-tab branch September 19, 2024 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants