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

Multiple Authors Access #324

Merged
merged 88 commits into from
Apr 17, 2024
Merged

Multiple Authors Access #324

merged 88 commits into from
Apr 17, 2024

Conversation

sorochak
Copy link
Contributor

@sorochak sorochak commented Feb 27, 2024

This PR introduces functionality to allow multiple users to have access to edit the same record. It addresses the feature request outlined in issue #78, aiming to enhance collaborative efforts and streamline the editing process on shared records. Only the original author of the record has the ability to share editing access with others. When a user logs in that has access to another user's records, they see a "Shared with me" link in the left sidebar that links to a Shared records page. They can edit but not submit or delete a record shared with them.

Changes Implemented:

  • Data Structure Modifications: Introduced a sharedWith node under each record for listing IDs of users with whom the record is shared. Also added a shares node at the regional level to map users to the records they're sharing.
  • Security Rule Updates: Revised security rules to support shared editing, ensuring that only authorized users can edit shared records.
  • Front End Enhancements:
    • Added a 'Shared with Me' section in the navigation drawer, conditionally rendered for users with shared records.
    • Implemented UI changes to allow users to share records by entering email addresses, mirroring the approach used in the contacts tab for consistency and user-friendliness.

@sorochak sorochak self-assigned this Feb 27, 2024
@sorochak sorochak linked an issue Feb 27, 2024 that may be closed by this pull request
Copy link

github-actions bot commented Feb 27, 2024

Visit the preview URL for this PR (updated for commit 1024ccc):

https://cioos-metadata-form--pr324-78-multiple-authors-pirm54tm.web.app

(expires Thu, 16 May 2024 22:47:40 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 57eda2a7622dc877ccadb675a0532261c52b09fd

@sorochak sorochak requested a review from fostermh February 27, 2024 22:35
@sorochak
Copy link
Contributor Author

Thank you for he input @JessyBarrette! It's now a searchable list of names.

@sorochak sorochak requested a review from fostermh April 16, 2024 22:42
Copy link

github-actions bot commented Apr 17, 2024

Visit the preview URL for this PR (updated for commit 1261175):

https://cioos-metadata-form-dev--pr324-78-multiple-authors-0684c7y6.web.app

(expires Fri, 17 May 2024 20:55:11 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 3ac34821b6d06862dcbab3b63feae62f4121aa78

…8-multiple-authors-access"

adjust config based on dev or prod project
@sorochak sorochak merged commit b9445fd into main Apr 17, 2024
3 checks passed
@sorochak sorochak deleted the 78-multiple-authors-access branch April 17, 2024 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

multiple authors access to edit same record
3 participants