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

Fix the quote command #527

Merged
merged 2 commits into from
Jun 20, 2022
Merged

Conversation

kleinesfilmroellchen
Copy link
Contributor

@kleinesfilmroellchen kleinesfilmroellchen commented Jun 19, 2022

That has been broken for a while, the reason comes down to Discord interaction restrictions.

Also a small fix to include quoted username in created PRs, which I find more transparent.

Fixes #356

Accessing GitHub APIs for PR creation can easily exceed 3s.
After that, the interaction ID is gone if we don't defer the reply.
That way, they can more easily see whether they have been quoted, which
is nice for giving consent.
@bgianfo
Copy link
Member

bgianfo commented Jun 20, 2022

I don't think this fixes the encoding/github issues that #356 describes.

Happy to merge these fixes as they look good, I just don't see how it addresses that specific bug.

@bgianfo bgianfo merged commit 5e10486 into SerenityOS:master Jun 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Quote Command Is Broken
2 participants