Paths

Table of Contentst

Extend pending thread IDs to support thick sidebars
ClosedPublic
Actions

Authored by marcin on Sep 6 2024, 12:28 PM.

Details

Reviewers

tomek
inka

Commits

rCOMMbbe775f0a586: Extend pending thread IDs to support thick sidebars

Summary

This differential extends pending thread IDs to support thick sidebars.

Test Plan

jest

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

marcin created this revision.Sep 6 2024, 12:28 PM

Herald added a subscriber: ashoat. · View Herald TranscriptSep 6 2024, 12:28 PM

Harbormaster completed remote builds in B31454: Diff 43919.Sep 6 2024, 12:55 PM

marcin requested review of this revision.Sep 6 2024, 12:55 PM

inka accepted this revision.Sep 6 2024, 1:07 PM

This revision is now accepted and ready to land.Sep 6 2024, 1:07 PM

Closed by commit rCOMMbbe775f0a586: Extend pending thread IDs to support thick sidebars. · Explain WhySep 6 2024, 1:11 PM

This revision was automatically updated to reflect the committed changes.

marcin added a commit: rCOMMbbe775f0a586: Extend pending thread IDs to support thick sidebars.

tomek added inline comments.Sep 9 2024, 8:09 AM

lib/shared/thread-utils.js
399	Wondering whether we should expose `thick` word in the user-facing place, the URL. How about `dm_sidebar`?

Revision Contents
Changeset List

Path

Size

lib/

shared/

thread-utils.js

32 lines

thread-utils.test.js

18 lines

utils/

validation-utils.js

2 lines

Diff 43923

View Options

lib/shared/thread-utils.js

View Options

lib/shared/thread-utils.test.js

View Options

lib/utils/validation-utils.js

Extend pending thread IDs to support thick sidebarsClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 43923

lib/shared/thread-utils.js

lib/shared/thread-utils.test.js

lib/utils/validation-utils.js

Extend pending thread IDs to support thick sidebars
ClosedPublic
Actions

Revision Contents
Changeset List