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

Add support for linking command to display a short link embed #26

Open
fredster33 opened this issue Jul 26, 2023 · 3 comments
Open

Add support for linking command to display a short link embed #26

fredster33 opened this issue Jul 26, 2023 · 3 comments

Comments

@fredster33
Copy link

It'd be nice if WikiBot could output an embed when it sends links. This allows people (especially lurkers) to get a basic understanding of what the page is about without needing to click on the link. Here's a mockup:

image

@stjohann
Copy link
Owner

stjohann commented Jul 26, 2023

That’s not something that will be done at any point because WikiBot can output any number of links that are in a message. Additionally, rendering these sorts of embeds takes more time and API requests, whereas current way WikiBot works doesn’t require those.

@stjohann
Copy link
Owner

stjohann commented Aug 1, 2023

Additionally, if we decided to use automatic Discord embeds like described in #25, they fail on links like https://ru.wikipedia.org/wiki/Википедия:Администраторы/Бывшие and display the content of https://ru.wikipedia.org/wiki/Бывшие instead.

Closing this issue as wontfix.

@stjohann stjohann closed this as not planned Won't fix, can't repro, duplicate, stale Aug 1, 2023
@stjohann
Copy link
Owner

Re-reading this issue again, I guess this can be done as a new command to display one link at a time like that. This would, at least, mean that people would use this type of links less, and it wouldn’t interfere with the standard way of wiki linking by the bot. Adjusting accordingly.

@stjohann stjohann reopened this Aug 19, 2023
@stjohann stjohann changed the title Use embeds Linking: Add support for !link command to display a short link embed Aug 19, 2023
@stjohann stjohann changed the title Linking: Add support for !link command to display a short link embed Add support for linking command to display a short link embed Aug 19, 2023
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

2 participants