downlords-faf-client icon indicating copy to clipboard operation
downlords-faf-client copied to clipboard

Support replay IDs in chat

Open micheljung opened this issue 9 years ago • 6 comments

People often enter replay IDs in chat, like #12345. Currently there is an IRC bot who recognizes this and posts a link in order to download the replay. Instead of this, the client should turn such IDs into links.

Whenever a user clicks on such a link, the client should open the replay in the replay vault.


Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

micheljung avatar Dec 11 '15 15:12 micheljung

Is there any way the client could detect its a proper replay id instead of just checking for /#\d+/? I guess, if possible, we should prevent creating links in messages like "I'm the #1 supernoob!" .

themasch avatar Jun 16 '17 06:06 themasch

Other than checking for whether a replay actually exists (but that should be the case for about numbers 1-3'000'000), we could have a different format like replay:1234 but that seems less convenient ;) And I think the false positives are negligible.

micheljung avatar Jun 18 '17 09:06 micheljung

Look at my PR i don't know why its not shown here, but i implemented a copy Link button and had replay Links being opened by the client and not the browser.

1-alex98 avatar Jun 18 '17 21:06 1-alex98

The issue ID needs to be linked in the PR description in order to show up here. Title is not enough.

micheljung avatar Jun 27 '17 08:06 micheljung

Can we close this since it's merged

andrewtvuong avatar Jun 12 '22 20:06 andrewtvuong

well I think it used to work but was removed again in some refactoring

1-alex98 avatar Jun 13 '22 11:06 1-alex98