Page MenuHomePhabricator
Differential D8486

[keyserver] Remove target blank
ClosedPublic
Actions

Authored by tomek on Jul 12 2023, 2:34 AM.
Tags
None
Referenced Files
F4494901: D8486.id.diff
Sat, Mar 1, 1:48 AM
Unknown Object (File)
Thu, Feb 13, 10:33 AM
Unknown Object (File)
Fri, Feb 7, 9:07 PM
Unknown Object (File)
Jan 13 2025, 1:01 PM
Unknown Object (File)
Jan 7 2025, 7:11 PM
Unknown Object (File)
Jan 7 2025, 7:11 PM
Unknown Object (File)
Jan 7 2025, 7:11 PM
Unknown Object (File)
Jan 7 2025, 7:08 PM
View All 75 Files
Subscribers
ashoat

Details

Reviewers
kamil
inka
Commits
rCOMM041958e44a76: [keyserver] Remove target blank
Summary

Unfortunately, it doesn't solve https://linear.app/comm/issue/ENG-4089/invite-links-dont-work-on-prod.

Test Plan

Check if clicking a link in Chrome doesn't open a new card.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tomek created this revision.Jul 12 2023, 2:34 AM
Herald added a subscriber: ashoat. · View Herald TranscriptJul 12 2023, 2:34 AM
Harbormaster completed remote builds in B20893: Diff 28615.Jul 12 2023, 2:52 AM
tomek requested review of this revision.Jul 12 2023, 2:52 AM
kamil accepted this revision.Jul 12 2023, 7:35 AM

Personally, I would revert D8465 instead of doing it manually but this also looks okay

This revision is now accepted and ready to land.Jul 12 2023, 7:35 AM
kamil added a comment.Jul 12 2023, 7:36 AM

ahh nvm, I see the difference - ignore my comment

Closed by commit rCOMM041958e44a76: [keyserver] Remove target blank (authored by tomek). · Explain WhyJul 13 2023, 2:03 AM
This revision was automatically updated to reflect the committed changes.
tomek added a commit: rCOMM041958e44a76: [keyserver] Remove target blank.

Revision Contents
Changeset List

PathSize
keyserver/
src/
responders/
6 lines

Diff 28652

View Options

keyserver/src/responders/website-responders.js

Loading...