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

[stable27] fix: Update file metadata only if file was changed #3231

Merged
merged 1 commit into from
Oct 18, 2023

Conversation

juliusknorr
Copy link
Member

@juliusknorr juliusknorr commented Oct 16, 2023

Track the state of the document modification status through the incoming post messages to make sure that we only update the mtime displayed in the file list if the file was actually changed.

@cypress
Copy link

cypress bot commented Oct 16, 2023

Passing run #666 ↗︎

0 31 0 0 Flakiness 0
⚠️ You've recorded test results over your free plan limit.
Upgrade your plan to view test results.

Details:

[stable27] fix: Update file metadata only if file was changed
Project: Richdocuments Commit: 256c124d93
Status: Passed Duration: 06:07 💡
Started: Oct 16, 2023 1:49 PM Ended: Oct 16, 2023 1:55 PM

Review all test suite changes for PR #3231 ↗︎

@juliusknorr juliusknorr changed the title fix: Update file metadata only if file was changed [stable27] fix: Update file metadata only if file was changed Oct 16, 2023
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliusknorr juliusknorr force-pushed the bugfix/noid/limit-mtime-update branch from db45238 to 256c124 Compare October 16, 2023 13:46
@juliusknorr juliusknorr added bug Something isn't working 3. to review Ready to be reviewed labels Oct 16, 2023
@juliusknorr juliusknorr merged commit 942b405 into stable27 Oct 18, 2023
35 checks passed
@juliusknorr juliusknorr deleted the bugfix/noid/limit-mtime-update branch October 18, 2023 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Ready to be reviewed bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants