-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
refactor(API): Refactor datasets API #2439
Conversation
Thank you for your PR. Would you please use English to write the source code comment? |
Of course, I will |
Thanks a lot. |
👌 |
@KevinHuSh Done. |
What problem does this PR solve?
discuss:#1102
Completed
TODO
I tried using this method, but it was not successful. It only allows token authentication when not logged in, but cannot skip token authentication when logged in 😢
I see some differences between the Web and SDK APIs, such as the key_mapping handling of the returned results. Until I figure it out, I cannot modify these codes to avoid causing more problems
Type of change