[lib] Update thread and keyserver ID regex
ClosedPublic
Actions

Authored by tomek on Wed, Aug 28, 8:20 AM.

Details

Reviewers

kamil
inka

Commits

rCOMM63f1eda390fd: [lib] Update thread and keyserver ID regex

Summary

This regex is used on web to recognize thread id in the url. When it isn't matched, we navigate to a default thread.

https://linear.app/comm/issue/ENG-9099/opening-a-link-to-a-thick-thread-resets-the-navigation-on-web

Test Plan

Open a web page on a thin thread that isn't the first, refresh the page and check if the same thread is still open.

Diff Detail

Repository

rCOMM Comm

Branch

master

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

tomek created this revision.Wed, Aug 28, 8:20 AM

Herald added a subscriber: ashoat. · View Herald TranscriptWed, Aug 28, 8:20 AM

Harbormaster returned this revision to the author for changes because remote builds failed.Wed, Aug 28, 8:29 AM

Harbormaster failed remote builds in B31346: Diff 43757!

Harbormaster completed remote builds in B31346: Diff 43757.Wed, Aug 28, 8:39 AM

tomek requested review of this revision.Wed, Aug 28, 8:39 AM

kamil accepted this revision.Wed, Aug 28, 9:34 AM

This revision is now accepted and ready to land.Wed, Aug 28, 9:34 AM

tomek added a child revision: D13202: [web] Keep an ID of a thick thread in the URL.Wed, Aug 28, 9:57 AM

Closed by commit rCOMM63f1eda390fd: [lib] Update thread and keyserver ID regex (authored by tomek). · Explain WhyWed, Aug 28, 10:00 AM

This revision was automatically updated to reflect the committed changes.

tomek added a commit: rCOMM63f1eda390fd: [lib] Update thread and keyserver ID regex.

Revision Contents
Changeset List

Path

Size

lib/

utils/

validation-utils.js

4 lines

Diff 43757

View Options

lib/utils/validation-utils.js

[lib] Update thread and keyserver ID regexClosedPublicActions