Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support replay IDs in chat #138

Closed
micheljung opened this issue Dec 11, 2015 · 6 comments
Closed

Support replay IDs in chat #138

micheljung opened this issue Dec 11, 2015 · 6 comments

Comments

@micheljung
Copy link
Member

micheljung commented Dec 11, 2015

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 micheljung added this to the v0.6.1-alpha milestone Dec 11, 2015
@micheljung micheljung removed the ready label Jan 26, 2016
@micheljung micheljung removed this from the v0.7.0-alpha milestone Jan 27, 2016
@micheljung micheljung changed the title Support replay URLs Support replay IDs in chat Dec 22, 2016
@themasch
Copy link
Contributor

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!" .

@micheljung
Copy link
Member Author

micheljung commented Jun 18, 2017

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.

@1-alex98
Copy link
Member

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.

@micheljung
Copy link
Member Author

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

@1-alex98 1-alex98 self-assigned this Oct 4, 2017
@andrewtvuong
Copy link

Can we close this since it's merged

@1-alex98
Copy link
Member

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

@1-alex98 1-alex98 removed their assignment Jul 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants