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: changed Spond.get_events() subgroupId Param to subGroupId #170

Merged
merged 1 commit into from
Feb 11, 2025

Conversation

SimonHenz97
Copy link
Contributor

After failing to restrict the event search to a specific subgroup, I realized that Spond's API uses subGroupId with a capital 'G' instead of subgroupId.

After reviewing this issue, I believe subGroupId is the correct parameter. However, due to certain circumstances, I'm currently unable to test whether my commit resolves the issue.

@elliot-100 elliot-100 self-requested a review February 11, 2025 11:53
Copy link
Collaborator

@elliot-100 elliot-100 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for spotting my error! It explains a few things. Verified with my live data.

@elliot-100 elliot-100 merged commit 81d8c2e into Olen:main Feb 11, 2025
5 checks passed
@elliot-100
Copy link
Collaborator

@Olen would you be OK to release a new version with this bugfix please?

@Olen
Copy link
Owner

Olen commented Feb 13, 2025

Sure. Give me some time to get home and do stuff

@elliot-100
Copy link
Collaborator

of course, no pressure!

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.

3 participants