Paths

Table of Contentst

Differential D12215

[lib] extend message store ops to handle processing local field ops
ClosedPublic
Actions

Authored by ginsu on May 25 2024, 4:31 AM.

Tags

None

Referenced Files

	F3494846: D12215.diff
	Thu, Dec 19, 6:57 AM

	Unknown Object (File)
	Wed, Dec 18, 6:34 AM

	Unknown Object (File)
	Wed, Dec 18, 6:34 AM

	Unknown Object (File)
	Wed, Dec 18, 6:34 AM

	Unknown Object (File)
	Wed, Dec 18, 6:34 AM

	Unknown Object (File)
	Wed, Dec 18, 6:34 AM

	Unknown Object (File)
	Tue, Nov 19, 1:55 PM

	Unknown Object (File)
	Nov 13 2024, 2:04 AM

View All 78 Files

Subscribers

Details

Reviewers

tomek
kamil
will

Commits

rCOMMa6595f319877: [lib] extend message store ops to handle processing local field ops

Summary

introduce necessary types + extend message store ops to handle processing local field using ops approach

Depends on D12214

Test Plan

flow + tested the entire stack at the end of the hackathon

Diff Detail

Repository

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ginsu created this revision.May 25 2024, 4:31 AM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptMay 25 2024, 4:31 AM

ginsu edited the summary of this revision. (Show Details)May 25 2024, 4:32 AM

ginsu added reviewers: tomek, kamil, will.

ginsu added a parent revision: D12214: [lib] reverse conditional logic for convertOpsToClientDBOps in message store ops.

Harbormaster completed remote builds in B29248: Diff 40654.May 25 2024, 4:58 AM

ginsu requested review of this revision.May 25 2024, 4:58 AM

kamil accepted this revision.May 27 2024, 2:55 AM

kamil added inline comments.

lib/types/message-types.js
507	`localMessageInfo` - the naming convention is inconsistent with e.g. `start_reached` but in other stores, we use camel case so probably it's fine. At least we can create a task to make it consistent someday but guessing this task will die in backlog and we'll never prioritize it

This revision is now accepted and ready to land.May 27 2024, 2:55 AM

will added a child revision: D12247: [sqlite] create message store local table.May 30 2024, 8:10 AM

ginsu added inline comments.Jun 5 2024, 10:16 AM

lib/types/message-types.js
507	Created this task to track https://linear.app/comm/issue/ENG-8373/update-fields-in-clientdbmessageinfo-and-clientdbthreadmessageinfo-to

rebase before landing

This revision was landed with ongoing or failed builds.Jun 5 2024, 10:26 AM

Closed by commit rCOMMa6595f319877: [lib] extend message store ops to handle processing local field ops (authored by ginsu). · Explain Why

This revision was automatically updated to reflect the committed changes.

ginsu added a commit: rCOMMa6595f319877: [lib] extend message store ops to handle processing local field ops.

Harbormaster completed remote builds in B29457: Diff 40987.Jun 5 2024, 10:34 AM

Revision Contents
Changeset List

Path

Size

lib/

ops/

message-store-ops.js

54 lines

types/

message-types.js

15 lines

Diff 40654

lib/ops/message-store-ops.js

Loading...

lib/types/message-types.js

Loading...