-
Notifications
You must be signed in to change notification settings - Fork 257
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
Document the "SDKs only" method waitForTask
#2171
Comments
@meilisearch/integration-team, is this still relevant? |
By using the wait_for-like methods the user should be aware of major performance bottlenecks they could face if used recklessly. So, it would be nice to document that in an official way, but I don't know how integrations-only content would fit in the docs these days 🤔. I also don't know about your other priorities, so it is up to you! I would classify this issue as a "nice to have" :) |
Yeah, so, a while ago (early 2023, pre-layoffs) I did some research on SDK documentation and it's a bit of a huge topic. We got two main takeaways from it:
Given how most of our SDKs are not even maintained by us nowadays, and how SDKs are not a strategic focus right now and for the foreseeable future, I'm closing this as not planned. |
An excellent context can be found here: meilisearch/integration-guides#243.
Also, I've discussed this privately with @dichotommy proposing the creation of a section in the docs where we can adequately document the usage of this kind of function.
The subject is this method, which is present in every SDK:
https://github.com/meilisearch/meilisearch-php/blob/main/src/Endpoints/Delegates/HandlesTasks.php#L38
What we expect to be present in the docs is:
CC: @meilisearch/integration-team (let me know if I miss anything)
The text was updated successfully, but these errors were encountered: