Page MenuHomePhabricator
Differential D13195

[lib] remove `newMemberIDs` from `DMThreadSettingsChanges`
ClosedPublic
Actions

Authored by kamil on Wed, Aug 28, 7:42 AM.
Tags
None
Referenced Files
F2788812: D13195.id43783.diff
Sat, Sep 21, 7:30 AM
F2788097: D13195.diff
Sat, Sep 21, 5:47 AM
Unknown Object (File)
Tue, Sep 17, 2:52 PM
Unknown Object (File)
Tue, Sep 17, 4:00 AM
Unknown Object (File)
Tue, Sep 17, 12:08 AM
Unknown Object (File)
Thu, Sep 12, 1:42 AM
Unknown Object (File)
Wed, Sep 11, 9:23 AM
Unknown Object (File)
Sat, Sep 7, 2:19 PM
View All 32 Files
Subscribers
ashoat

Details

Reviewers
tomek
inka
marcin
Commits
rCOMM92cd9357bc28: [lib] remove `newMemberIDs` from `DMThreadSettingsChanges`
Summary

ENG-8979;

For adding new members there is separate spec, ADD_MEMBERS and ADD_VIEWER_TO_THREAD_MEMBERS.

Depends on D13192

Test Plan

Flow

Diff Detail

Repository
rCOMM Comm
Branch
logic-4
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil created this revision.Wed, Aug 28, 7:42 AM
kamil held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptWed, Aug 28, 7:42 AM
kamil edited the summary of this revision. (Show Details)Wed, Aug 28, 7:43 AM
kamil added a parent revision: D13192: [lib] implement changing DM threads settings.
kamil published this revision for review.Wed, Aug 28, 7:55 AM
Harbormaster completed remote builds in B31340: Diff 43751.Wed, Aug 28, 7:59 AM
tomek accepted this revision.Wed, Aug 28, 9:19 AM
tomek added inline comments.
lib/shared/dm-ops/change-thread-settings-spec.js
60

This function could now be inlined in addMembersSpec.

154

This can also be inlined.

lib/types/dm-ops.js
31–32

This can be deleted.

281–299

We can inline dmChangeThreadSettingsBaseValidatorShape into dmChangeThreadSettingsOperationValidator.

306–316

We can delete these.

This revision is now accepted and ready to land.Wed, Aug 28, 9:19 AM
kamil updated this revision to Diff 43778.Thu, Aug 29, 12:20 AM

address review

Harbormaster completed remote builds in B31358: Diff 43778.Thu, Aug 29, 12:40 AM
Closed by commit rCOMM92cd9357bc28: [lib] remove `newMemberIDs` from `DMThreadSettingsChanges` (authored by kamil). · Explain WhyThu, Aug 29, 1:16 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMM92cd9357bc28: [lib] remove `newMemberIDs` from `DMThreadSettingsChanges`.

Revision Contents
Changeset List

Diff 43751

View Options

lib/actions/thread-actions.js

Loading...
View Options

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

Loading...
View Options

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

Loading...
View Options

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

Loading...
View Options

lib/types/dm-ops.js

Loading...