Page MenuHomePhabricator
Differential D8744

[lib] Handle new id schema in thread regexes
ClosedPublic
Actions

Authored by michal on Aug 7 2023, 2:38 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jun 11, 8:27 PM
Unknown Object (File)
Tue, Jun 11, 4:09 PM
Unknown Object (File)
Mon, Jun 10, 7:24 PM
Unknown Object (File)
Thu, Jun 6, 1:36 AM
Unknown Object (File)
May 6 2024, 9:38 PM
Unknown Object (File)
May 4 2024, 2:38 PM
Unknown Object (File)
Apr 28 2024, 10:55 PM
Unknown Object (File)
Apr 8 2024, 9:34 PM
View All 31 Files
Subscribers
ashoat

Details

Reviewers
tomek
ginsu
rohan
Commits
rCOMMc054a1e8ba3f: [lib] Handle new id schema in thread regexes
Summary

ENG-4586

Updates threadRegex in url-utils.js and pendingThreadIDRegex to handle both old and new id schemas.

Test Plan
  • checked if navigating back and forth works on web and on desktop
  • searched for regex in our codebase and checked if there are any more instances that need to be updated
  • yarn test

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
lib/
shared/
4 lines
9 lines
utils/
3 lines
3 lines
18 lines

Diff 29580

View Options

lib/shared/thread-utils.js

Loading...
View Options

lib/shared/thread-utils.test.js

Loading...
View Options

lib/utils/url-utils.js

Loading...
View Options

lib/utils/validation-utils.js

Loading...
View Options

lib/utils/validation-utils.test.js

Loading...