-
Notifications
You must be signed in to change notification settings - Fork 111
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
📚 Documentation: library code does not contain clear and or concise documentation #152
Comments
@folaoluwafemi Thank you for your suggestion. We will work on it. |
Whoever works on this can refer to the SDK for web or node to see what the comment docs should be. |
Hey, @stnguyen90 @lohanidamodar I'm new to Appwrite community. I just saw this issue and can contribute to clarifying the docs. May I give it a go? |
This isn't really a docs issue, this is a SDK generator issue I think, right @stnguyen90? But this does sound like a good idea, I believe we brought this up during Hacktoberfest last year? |
So what can we do to solve this? I really don't know what you are talking about @gewenyu99 |
We would have to make changes to the SDK Generator. |
@iswarmondal, for example, files like these: would need to be updated. |
@stnguyen90 thanks mate, 🙌🏼 |
@iswarmondal, actually, we need to update this ASAP, so I think we're going to work on it internally. |
@stnguyen90 no problem, I'm happy to interact with you guys |
@folaoluwafemi With new version 9.0.1 this issue should be resolved. We have tried to cover all public functions. Feel free to reopen the issue if you think there's still things missing. |
💭 Description
Basically the docs from the appwrite.io/docs page are clear, but I expect more finer grained docs from the flutter/dart sdk code to make it much more easier to understand what to do, let me paint a scenario.
main.dart
file.Client
class [from appwrite lib] I see a paramselfSigned
and ask myself what it meansctrl + j
on AS or Intellij) to show info from the added documentation, and nothing that can help my situation comes upMy point? it would have been incredibly helpful to have the docs from the appwrite lib, even if it's just the public facing part of the lib.
Thank you.
👀 Have you spent some time to check if this issue has been raised before?
🏢 Have you read the Code of Conduct?
The text was updated successfully, but these errors were encountered: