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

Adds interaction handler class #76

Merged
merged 3 commits into from
Feb 24, 2024
Merged

Adds interaction handler class #76

merged 3 commits into from
Feb 24, 2024

Conversation

Skelmis
Copy link
Member

@Skelmis Skelmis commented Feb 24, 2024

Summary

Adds an interaction handler class for easier work going forward

Checklist

  • Has been manually tested
  • Has unit-tests, if applicable
  • New features have attached cooldowns
  • Any new strings are localized correctly
  • These work on both new and old style suggestions
  • All interactions have been deferred
  • New errors have been updated on stats.suggestions.gg
  • Guild config method names aren't duplicated
  • New localizations have been added
  • Documentation on docs.suggestions.gg has been updated

Skelmis and others added 3 commits September 28, 2023 13:45
* feat: add InteractionHandler class

* feat: add tests for the class

* chore: run tests on all branches

* chore: make actions nicer to remove duplicate runs

* fix: tests

* fix: tests
@Skelmis Skelmis marked this pull request as ready for review February 24, 2024 05:40
@Skelmis Skelmis merged commit cee879e into feat/version/3.22 Feb 24, 2024
1 check failed
@Skelmis Skelmis deleted the refactor/base branch February 24, 2024 05:40
Skelmis added a commit that referenced this pull request Mar 11, 2024
* chore: initial commit

* Adds interaction handler class (#76)

* chore: change readme

* feat: add InteractionHandler class (#64)

* feat: add InteractionHandler class

* feat: add tests for the class

* chore: run tests on all branches

* chore: make actions nicer to remove duplicate runs

* fix: tests

* fix: tests

* fix: remove information disclosure via queue approval (#78)

* fix: suggestion message race condition

* Update pt_BR.json to version 3.20 (#68)

* Update pt_BR.json to version 3.20

* Update PULL_REQUEST_TEMPLATE.md

* Suggestion queue update

---------

Co-authored-by: Ethan <[email protected]>

* fix: json file format

* fix: images 404'ing (#79)

* feat: physical suggestions queue (#77)

* fix: InteractionHandler tests

* feat: move current suggestions queue to use interaction handler

* feat: port existing queue rejection/approval to new generic method

* fix: virtual queue approval and rejection system

* feat: add initial physical queue

* fix: messages being sent

* fix: error message for missing queue channel

* feat: add items to /config get

* fix: tests

* chore: make button styling more obvious

* chore: rename physical queue to channel queue

* chore: fix wording

* feat: move to logoo logging (#80)

* feat: start logoo integration

* chore: remove un-needed metadata

* feat: migrate to Logoo

* chore: shush httpx logs

* feat: add bot_version to logs

* chore: bump logoo version

---------

Co-authored-by: Davi <[email protected]>
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.

1 participant