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 7TV exception when streamer has no emotes #2448

Merged
merged 6 commits into from
May 2, 2023

Commits on May 1, 2023

  1. Add "integration" tests for testing responses to the 7TV API

    These fail atm, and can be run by setting the
    `PAJBOT_RUN_INTEGRATION_TESTS` environment variable to `1`
    pajlada committed May 1, 2023
    Configuration menu
    Copy the full SHA
    53bb35c View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    3b4c850 View commit details
    Browse the repository at this point in the history
  3. typecheck/model 7TV API requests, fixes some edge-cases

    edge-cases include user who logged into 7tv but doesn't have an emote
    set, or user who logged into 7tv, created an emote set, but didn't add
    any emotes
    pajlada committed May 1, 2023
    Configuration menu
    Copy the full SHA
    b181d80 View commit details
    Browse the repository at this point in the history
  4. Add changelog entry

    pajlada committed May 1, 2023
    Configuration menu
    Copy the full SHA
    d576e77 View commit details
    Browse the repository at this point in the history

Commits on May 2, 2023

  1. reformat

    pajlada committed May 2, 2023
    Configuration menu
    Copy the full SHA
    0708299 View commit details
    Browse the repository at this point in the history
  2. Merge remote-tracking branch 'origin/master' into fix/7tv-exception-w…

    …hen-streamer-has-no-emotes
    pajlada committed May 2, 2023
    Configuration menu
    Copy the full SHA
    ac7d264 View commit details
    Browse the repository at this point in the history