mirror of
https://github.com/ramsayleung/rspotify.git
synced 2026-04-25 23:45:52 +03:00
[GH-ISSUE #378] Make "private" methods in BaseClient public #119
Labels
No labels
Stale
bug
discussion
enhancement
good first issue
good first issue
help wanted
pull-request
question
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/rspotify#119
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @marioortizmanero on GitHub (Dec 29, 2022).
Original GitHub issue: https://github.com/ramsayleung/rspotify/issues/378
Is your feature request related to a problem? Please describe.
We currently have advanced methods in
BaseClientlikeapi_get,api_post, etc (see #356 to learn more about the new naming). However, they aren't properly documented, and making them public would make it possible to implement custom endpoints. For example, users would be able to implement #104 themselves, which we currently don't support.Describe the solution you'd like
Make all the
#[doc(hidden)]methods inBaseClientandOAuthClientpublic and document them appropriately. Indicate that they aren't meant to be used directly, but include an example of how users could implement their own custom endpoints. We could have a file in theexamplesdirectory for that.Describe alternatives you've considered
We currently have them as private, but it's just a hack. Users can still access them, we just hide the methods from the docs, which might be confusing. Besides, having them public can be useful.
@github-actions[bot] commented on GitHub (Jun 24, 2023):
Message to comment on stale issues. If none provided, will not mark issues stale
RSPOTIFY_#255