[lib] check timestamps before processing delete entry operation
ClosedPublic
Actions

Authored by will on Sep 16 2024, 1:33 AM.

Details

Reviewers

marcin
ashoat
tomek

Commits

rCOMM24e495a0bb66: [lib] check timestamps before processing delete entry operation

Summary

Checks timestamps to unsure delete operation isn't alread out of date. If it is, produces a no-op update.

Depends on D13347

Test Plan

As previous, unsure yet how to test this. Will update test plan later

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

will created this revision.Sep 16 2024, 1:33 AM

will added a child revision: D13349: [web] disable history button on web on clicking entry.

Harbormaster completed remote builds in B31630: Diff 44202.Sep 16 2024, 2:09 AM

will requested review of this revision.Sep 16 2024, 2:09 AM

will mentioned this in D13340: [lib] DMOperationSpec for delete entry operation.Sep 16 2024, 5:41 AM

tomek accepted this revision.Sep 16 2024, 5:49 AM

tomek added inline comments.

lib/shared/dm-ops/delete-entry-spec.js
64–65	Can we exit early if this condition isn't met and use the same property name (`time`) consistently for all the operations?

This revision is now accepted and ready to land.Sep 16 2024, 5:49 AM

Closed by commit rCOMM24e495a0bb66: [lib] check timestamps before processing delete entry operation (authored by will). · Explain WhySep 16 2024, 2:27 PM

This revision was automatically updated to reflect the committed changes.

will added a commit: rCOMM24e495a0bb66: [lib] check timestamps before processing delete entry operation.

Revision Contents
Changeset List

Path

Size

lib/

shared/

dm-ops/

delete-entry-spec.js

43 lines

Diff 44202

View Options

lib/shared/dm-ops/delete-entry-spec.js

[lib] check timestamps before processing delete entry operationClosedPublicActions