-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Disable-readers-word-edit #393
Conversation
✅ Deploy Preview for dailp ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One point for discussion, but this looks good.
We want the above functionality in order to support in-person work teams who prefer working | ||
on paper or through discussion, then putting everything in the computer at once | ||
via one DAILP user account. */ | ||
let userRole = useUserRole() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(I think it's out of scope for this ticket but we should still think about it) Should this be done with groups and not with roles? Ie. the important thing is having a group (you may have others) not being a particular role (you only have one)? We haven't really specified how any of this should work, but I think it's worth raising for thought.
Word panel tsx: Trim panel previews and make sure that word panel and paragraph panel options are in their appropriate places. Review stale code in the word panel and paragraph panel. Separate the common panel from the word panel file. |
No description provided.