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: required activities field is not being sent in presence update payload #557

Merged
merged 1 commit into from
Sep 20, 2023
Merged

Conversation

priyanuj-gogoi
Copy link
Contributor

Description

Fixes an issue where the bot fails to connect the gateway and logs Unresumable invalid session when PresenceBuilder() activities is ignored.

PresenceBuilder(
  since: DateTime.now(),
  // activities: []
  status: CustomUserStatus.online,
  isAfk: false
);

When sending a presence update event payload, the activities field is supposed to send. Check the docs, there is no ? after activities.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • Ran dart analyze or make analyze and fixed all issues
  • Ran dart format --set-exit-if-changed -l 160 ./lib or make format and fixed all issues
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works

@Lexedia Lexedia added the bug Something isn't working label Sep 20, 2023
@Lexedia Lexedia added this to the 6.0 milestone Sep 20, 2023
@Lexedia Lexedia merged commit 650ee17 into nyxx-discord:next Sep 20, 2023
3 checks passed
@priyanuj-gogoi priyanuj-gogoi deleted the next branch September 24, 2023 04:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants