Page MenuHomePhabricator
Differential D13349

[web] disable history button on web on clicking entry
ClosedPublic
Actions

Authored by will on Sep 16 2024, 1:34 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 9, 6:56 PM
Unknown Object (File)
Wed, Oct 23, 8:41 PM
Unknown Object (File)
Wed, Oct 23, 9:12 AM
Unknown Object (File)
Wed, Oct 23, 9:12 AM
Unknown Object (File)
Wed, Oct 23, 9:11 AM
Unknown Object (File)
Oct 19 2024, 2:17 PM
Unknown Object (File)
Oct 18 2024, 1:58 PM
Unknown Object (File)
Sep 29 2024, 9:24 AM
View All 26 Files
Subscribers
None

Details

Reviewers
tomek
ashoat
marcin
Commits
rCOMM7b68beb6c9d3: [web] disable history button on web on clicking entry
Summary

This disables the history button when clicking a given entry if thick threaded on web. Disabling the history button for a day box will be introduced in a separate diff

Depends on D13348

Test Plan

Checked on local web app. history button no longer appears

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

will created this revision.Sep 16 2024, 1:34 AM
will added a child revision: D13350: [native][web] use tunnelbroker for online status of thick threaded entries instead of keyserver.
Harbormaster completed remote builds in B31631: Diff 44203.Sep 16 2024, 2:14 AM
will requested review of this revision.Sep 16 2024, 2:14 AM
tomek accepted this revision.Sep 16 2024, 5:50 AM

Let's document this somewhere. Also, it might be a good idea to introduce a single flag for it.

This revision is now accepted and ready to land.Sep 16 2024, 5:50 AM
ashoat added a comment.Sep 16 2024, 1:04 PM

Where would we document? Maybe @tomek was thinking of something like a function function isCalendarEntryHistoryEnabled(threadType: ThreadType): boolean

ashoat mentioned this in D13354: [web] Disable thick thread entries from showing up in the history modal as history entries.Sep 16 2024, 1:23 PM
Closed by commit rCOMM7b68beb6c9d3: [web] disable history button on web on clicking entry (authored by will). · Explain WhySep 16 2024, 2:27 PM
This revision was automatically updated to reflect the committed changes.
will added a commit: rCOMM7b68beb6c9d3: [web] disable history button on web on clicking entry.
tomek added a comment.Sep 17 2024, 1:01 AM
In D13349#376126, @ashoat wrote:

Where would we document? Maybe @tomek was thinking of something like a function function isCalendarEntryHistoryEnabled(threadType: ThreadType): boolean

Ah, I should've added more details. I was thinking about creating a task that tracks introducing the history of thick thread entries and removing a flag (or a function).

Revision Contents
Changeset List

PathSize
web/
calendar/
5 lines

Diff 44203

View Options

web/calendar/entry.react.js

Loading...