-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Update public_api.md (changes made) #2712
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
Conversation
🎉 Thanks for opening this pull request! Please check out our contributing guidelines if you haven't already. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nicely Done 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks so much for working on this, this is great so far!
I think one change that I feel would make more sense is for the added section "Enabling Access Token UI in Settings Page" to be higher up in the document, such as in between the sections "Models" and "Sketch".
Did some changes, is it good to go? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for working on this!
Fixes #1769
Changes:
I have verified that this pull request:
npm run lint
)npm run test
)develop
branch.Fixes #123