-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
[Concept Entry] Python Plotly #4609
[Concept Entry] Python Plotly #4609
Conversation
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.
Hey @joseandersonramos! Thank you for contributing to the Codecameny Docs!
The entry is well written!
Sorry for the inconvenience caused, the issue mentions that it needs an example
block, but I think that is not needed. Please follow the format as follows:
(1) Metadata
(2) A short description of plotly
(3) Installation (You can remove the syntax
block and add the installation part as suggested in the entry)
(4) A short description of the Plotly modules
(very short). (You can remove the example
block and add this).
I have added some suggestions, please review and modify them at your earliest convenience.
Lastly, please update the path to: docs/content/plotly/plotly.md
Thank you!
Received!, Thank you for the suggestion! |
Co-authored-by: Mamta Wardhani <53176352+mamtawardhani@users.noreply.github.com>
Co-authored-by: Mamta Wardhani <53176352+mamtawardhani@users.noreply.github.com>
Co-authored-by: Mamta Wardhani <53176352+mamtawardhani@users.noreply.github.com>
Co-authored-by: Mamta Wardhani <53176352+mamtawardhani@users.noreply.github.com>
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.
Hello @joseandersonramos! I've suggested a few changes in the phrasing, please review and modify those at your earliest convenience.
Thank you!
Co-authored-by: Mamta Wardhani <53176352+mamtawardhani@users.noreply.github.com>
Co-authored-by: Mamta Wardhani <53176352+mamtawardhani@users.noreply.github.com>
Co-authored-by: Mamta Wardhani <53176352+mamtawardhani@users.noreply.github.com>
Co-authored-by: Mamta Wardhani <53176352+mamtawardhani@users.noreply.github.com>
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.
Looks good to me! Good for the next review!
Hello @Sriparno08, |
No, @joseandersonramos. There's nothing needed from your side for now. |
Ok, thank you. |
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.
Looks good, @joseandersonramos!
👋 @joseandersonramos 🎉 Your contribution(s) can be seen here: https://www.codecademy.com/resources/docs/plotly Please note it may take a little while for changes to become visible. |
Description
Issue Solved
Type of Change
Checklist
main
branch.Issues Solved
section.