Page MenuHomePhabricator
Differential D13354

[web] Disable thick thread entries from showing up in the history modal as history entries
ClosedPublic
Actions

Authored by will on Mon, Sep 16, 1:20 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Sep 17, 12:47 PM
Unknown Object (File)
Tue, Sep 17, 12:06 PM
Unknown Object (File)
Tue, Sep 17, 11:41 AM
Unknown Object (File)
Tue, Sep 17, 10:41 AM
Unknown Object (File)
Tue, Sep 17, 8:50 AM
Unknown Object (File)
Tue, Sep 17, 6:41 AM
Unknown Object (File)
Tue, Sep 17, 5:59 AM
Unknown Object (File)
Tue, Sep 17, 5:58 AM
View All 13 Files
Subscribers
None

Details

Reviewers
ashoat
tomek
marcin
Commits
rCOMM6348d732c201: [web] Disable thick thread entries from showing up in the history modal as…
Summary

This disables thick thread entries from showing up in the history modal

Depends on D13350

Test Plan

Tested on local webapp. Keyserver history entries still showed up fine but thick thread entries did not

Diff Detail

Repository
rCOMM Comm
Branch
land_thick_thread_entries
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

will created this revision.Mon, Sep 16, 1:20 PM
ashoat published this revision for review.Mon, Sep 16, 1:23 PM
ashoat added inline comments.
web/modals/history/history-modal.react.js
116–117

Shorthand

ashoat accepted this revision.Mon, Sep 16, 1:23 PM

Before landing, can you create a follow-up task to create a flag that can gate both this and D13349?

This revision is now accepted and ready to land.Mon, Sep 16, 1:23 PM
Harbormaster completed remote builds in B31647: Diff 44235.Mon, Sep 16, 1:37 PM
will updated this revision to Diff 44243.Mon, Sep 16, 2:08 PM

final rebase

Harbormaster completed remote builds in B31655: Diff 44243.Mon, Sep 16, 2:25 PM
Closed by commit rCOMM6348d732c201: [web] Disable thick thread entries from showing up in the history modal as… (authored by will). · Explain WhyMon, Sep 16, 2:27 PM
This revision was automatically updated to reflect the committed changes.
will added a commit: rCOMM6348d732c201: [web] Disable thick thread entries from showing up in the history modal as….
ashoat added a comment.Mon, Sep 16, 2:36 PM

@will created a follow-up task, but didn't document it here. Here it is: https://linear.app/comm/issue/ENG-9302/create-a-flag-for-enabling-history-on-thick-threaded-entries

Revision Contents
Changeset List

PathSize
web/
modals/
history/
10 lines

Diff 44235

View Options

web/modals/history/history-modal.react.js

Loading...