-
Notifications
You must be signed in to change notification settings - Fork 9
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: APP-413 Show project admin from offchain data on project page #2584
Conversation
✅ Deploy Preview for terrasos ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for regen-website ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@erikalogie @S4mmyb see testing instructions |
I think we should remove it for off-chain projects if we keep that text. Alternatively (which might actually be better), we update the tooltip text on both to something like, "The entity responsible for technical management, editing project content, and managing participant access for contributions and updates." |
I like using the same tooltip text on both, maybe we can shorten your suggestion just a bit like so: " The entity responsible for technical management, content editing, and managing participant access for contributions." |
That's perfect! |
97307a5
to
6df2ba4
Compare
@S4mmyb @erikalogie I've updated the project admin tooltip |
@blushi LGTM |
+1, thanks @blushi |
6df2ba4
to
a36993e
Compare
Description
https://regennetwork.atlassian.net/browse/APP-413
Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...
How to test
Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...