mattermost-community / focalboard

Focalboard is an open source, self-hosted alternative to Trello, Notion, and Asana.
https://www.focalboard.com
Other
22.02k stars 1.99k forks source link

prevent ModifiedBy overwrite #4907

Open omggga opened 1 year ago

omggga commented 1 year ago

Summary

This pull request introduces a fix to ensure that the ModifiedBy field is not overwritten if it already contains a userID. We added a conditional check to verify if the ModifiedBy field is empty before assigning a new userID, enhancing the data integrity of the block modification tracking.

By default the system uses default userID from person, who is importing data. If board.jsonl is already filled with existing IDs from current board or mattermost instance, we should accept it, not overwrite.

Ticket Link

https://community.mattermost.com/core/pl/7x9n4kfy4pbdxphsnuhtoodqxy

mattermost-build commented 1 year ago

Hello @omggga,

Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

mattermost-build commented 1 year ago

This PR has been automatically labelled "stale" because it hasn't had recent activity. A core team member will check in on the status of the PR to help with questions. Thank you for your contribution!