Skip to content

feat: adapt to v1#29

Closed
null8626 wants to merge 21 commits intoTop-gg-Community:masterfrom
null8626:master
Closed

feat: adapt to v1#29
null8626 wants to merge 21 commits intoTop-gg-Community:masterfrom
null8626:master

Conversation

@null8626
Copy link
Member

@null8626 null8626 commented Jun 16, 2025

Closes #28

The following pull request does the following:

  • Adapts the C# SDK to v0 by adding features such as reviews and removing many legacy features. (Note that the search feature in GET /bots which will be removed in v1 as per Veld, is also removed.)
  • Merges the AuthenticatedBotListApi class with the DiscordBotListApi class as handling unauthenticated requests is very much pointless (very few endpoints are free to use without an API token.) Now everything is done through DiscordBotListApi.
  • Merges the SmallWidgetOptions class with the LargeWidgetOptions class. Now everything is done through Widget.
  • Implements a proper autoposter/SubmissionAdapter.
  • Bumps dependencies.
  • Updates outdated GitHub URLs and LICENSE copyright year.

As of v0, everything has been tested.

@null8626
Copy link
Member Author

Back to draft as Veld is currently working on widgets.

@null8626 null8626 marked this pull request as draft June 17, 2025 06:20
@null8626
Copy link
Member Author

Veld recently said that v0 is deprecated, which means that the v0 pull requests will now be delayed to facilitate changes bound in v1, sorry!

@null8626 null8626 changed the title feat: adapt to v0 feat: adapt to v1 Jun 17, 2025
@null8626 null8626 force-pushed the master branch 4 times, most recently from b7c596a to 6989a51 Compare June 20, 2025 04:20
@null8626 null8626 force-pushed the master branch 5 times, most recently from 0738402 to fc21e55 Compare June 20, 2025 07:49
@null8626 null8626 force-pushed the master branch 3 times, most recently from 422705d to d732006 Compare June 23, 2025 05:15
@null8626 null8626 force-pushed the master branch 5 times, most recently from f73ac93 to 42549d2 Compare June 25, 2025 11:30
@null8626 null8626 force-pushed the master branch 6 times, most recently from 79987f9 to a48f96b Compare June 27, 2025 12:40
@null8626
Copy link
Member Author

Closed, will split this into multiple pull requests.

@null8626 null8626 closed this Sep 16, 2025
@null8626 null8626 mentioned this pull request Sep 29, 2025
null8626 added a commit to null8626/dotnet-sdk that referenced this pull request Sep 29, 2025
@null8626 null8626 mentioned this pull request Oct 2, 2025
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.

2 participants