Skip to content
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

Improve content synchronization #7

Merged
merged 3 commits into from
Aug 16, 2023

Conversation

schurzi
Copy link

@schurzi schurzi commented May 3, 2023

These changes reduces mailspam when updating pages and attachments in Confluence.

Without the minorEdit parameter every change creates a mail for all watchers. The default is now changed to use minorEdit=True and thus not create notification mails. The previous behaviour can be restored by using the new cli option --no-minoredit.

Another implemented optimization is focused on attachments. The previous implementation was syncing all attachments everytime. When a page has a lot of attachments, this quickly creates many versions and wastes storage. The optimization uses the comment field to store a hash of the uploaded file and only re-uploads the contetns, if the hash of the current file does not match the one stored in comment. This behaviour can also be disabled by specifying --no-optimizeattachments .

schurzi added 3 commits May 3, 2023 21:00
Signed-off-by: Martin Schurz <Martin.Schurz@t-systems.com>
Signed-off-by: Martin Schurz <Martin.Schurz@t-systems.com>
Signed-off-by: Martin Schurz <Martin.Schurz@t-systems.com>
@mrporcles
Copy link
Collaborator

Hi and sorry for the extremely delayed response. Your PR and all submissions to this project are very much appreciated. I am just doing a quick review and then will look to merge. Thank you again for your contributions.

@mrporcles mrporcles merged commit 2ef1f4a into vmware-tanzu-labs:master Aug 16, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants