mattermost / focalboard

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

prevent ModifiedBy overwrite #4907

Open omggga opened 8 months ago

omggga commented 8 months 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 8 months 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 8 months 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!