-
Notifications
You must be signed in to change notification settings - Fork 3
feat: adapt to newer changes of v0 #13
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
base: main
Are you sure you want to change the base?
Conversation
|
For your information: This branch is already tested. |
topgg.rc
Outdated
| VALUE "FileVersion", "3.0.0" | ||
| VALUE "ProductVersion", "3.0.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this a major version? did the library break for devs?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some changes here are breaking, like:
- Fetching bot stats now only returns the server count as shards are deprecated.
- The ability to fetch Top.gg users (
get_user) is removed. - The minimum supported version for building the library is increased from
3.8.2to3.15.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Most breaking changes reverted, with the exception of the CMake version (tried using an older version, posting stats did not work.)
The latest version is now 2.1.0 instead of 3.0.0.
The following pull request is a toned down version of #12. This pull request focuses solely on the
changes done to the SDK in the v0 of the API. Such as:
LICENSE.Other changes will be issued in subsequent pull requests once this pull request is merged.