Page MenuHomePhabricator
Differential D13573

[lib] remove checking for message IDs collision
ClosedPublic
Actions

Authored by kamil on Wed, Oct 2, 6:25 AM.
Tags
None
Referenced Files
F2871986: D13573.id44825.diff
Wed, Oct 2, 9:34 PM
F2871959: D13573.id44824.diff
Wed, Oct 2, 9:30 PM
F2870849: D13573.diff
Wed, Oct 2, 7:48 PM
F2866567: D13573.id44825.diff
Wed, Oct 2, 11:35 AM
F2866566: D13573.id44824.diff
Wed, Oct 2, 11:35 AM
F2866530: D13573.id.diff
Wed, Oct 2, 11:35 AM
F2866523: D13573.diff
Wed, Oct 2, 11:33 AM
Subscribers
None

Details

Reviewers
ashoat
tomek
Commits
rCOMMfb674c2f263b: [lib] remove checking for message IDs collision
Summary

Short term fix to: ENG-9464.

There is a comment in ENG-9468 to came up with correct solution.

Test Plan

On the master:

  1. Open physical device
  2. In DM thread send a test message
  3. Notif delivered
  4. Message not visible in UI
  5. Seeing log: Discarded a send_text_message operation because message with the same ID 27caa155-d435-4ef8-9d8e-5b0862861e02 already exists in the store
  6. Kill/restart the app - message is visible regardless of network/TB connection.

Apply this diff:

  1. Open physical device
  2. In DM thread send a test message
  3. Notif delivered
  4. Message immediately visible in UI
  5. Not seeing any log about the discarded operation.

Diff Detail

Repository
rCOMM Comm
Branch
fix-validation
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil created this revision.Wed, Oct 2, 6:25 AM
kamil held this revision as a draft.
kamil published this revision for review.Wed, Oct 2, 6:27 AM
kamil edited the test plan for this revision. (Show Details)
ashoat accepted this revision.Wed, Oct 2, 6:32 AM
This revision is now accepted and ready to land.Wed, Oct 2, 6:32 AM
Harbormaster completed remote builds in B32015: Diff 44824.Wed, Oct 2, 6:43 AM
Closed by commit rCOMMfb674c2f263b: [lib] remove checking for message IDs collision (authored by kamil). · Explain WhyWed, Oct 2, 6:52 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMMfb674c2f263b: [lib] remove checking for message IDs collision.

Revision Contents
Changeset List

PathSize
lib/
shared/
dm-ops/
36 lines
39 lines
35 lines
14 lines

Diff 44824

View Options

lib/shared/dm-ops/change-thread-settings-spec.js

Loading...
View Options

lib/shared/dm-ops/create-sidebar-spec.js

Loading...
View Options

lib/shared/dm-ops/dm-op-utils.js

Loading...
View Options

lib/shared/dm-ops/process-dm-ops.js

Loading...