Page MenuHomePhabricator

[web] Allow copying the link
ClosedPublic

Authored by tomek on Jun 16 2023, 4:33 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 29, 4:45 PM
Unknown Object (File)
Tue, Nov 26, 6:26 AM
Unknown Object (File)
Tue, Nov 26, 4:34 AM
Unknown Object (File)
Sat, Nov 23, 7:03 AM
Unknown Object (File)
Sun, Nov 10, 12:32 PM
Unknown Object (File)
Sat, Nov 9, 4:04 AM
Unknown Object (File)
Fri, Nov 8, 2:51 PM
Unknown Object (File)
Fri, Nov 8, 8:03 AM
Subscribers

Details

Summary

Call clipboard copy API just like in other places on web. The discussion about operation feedback is still in progress https://www.figma.com/file/a1nkbWgbgjRlrOY9LVurTz?node-id=11400:282507#473816740.

Depends on D8238

Test Plan

Click copy button and check if the link is in the clipboard.

Diff Detail

Repository
rCOMM Comm
Branch
invite-web
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

tomek requested review of this revision.Jun 16 2023, 4:54 AM
web/invite-links/view-invite-link-modal.react.js
28

Are there other places where we generate this link? I'm a fan of having a single source of truth for such values somewhere in lib/ to make it easier to change where we e.g. migrate to a new domain.

I did sth similar in lib/utils/blob-service.js. What do you think?

web/invite-links/view-invite-link-modal.react.js
28

I see it is hardcoded in two places on native. Can we define this somewhere in lib?

web/invite-links/view-invite-link-modal.react.js
28

Yep, that makes sense!

Looks good but remember to define the URL in one place

This revision is now accepted and ready to land.Jun 22 2023, 3:23 AM

Rebase - use extracted function

This revision was automatically updated to reflect the committed changes.