Docstring generator is wrong for certain properties including sort_by #1205

Activity

1. Marc-Andre-Rivet commented on Apr 21, 2020

   

   Marc-Andre-Rivet

   on Apr 21, 2020

   Contributor

   More actions

   These entries are created by the generator, moving the issue to the Dash repo.
2. Marc-Andre-Rivet

   transferred this issue fromplotly/dash-tableon Apr 21, 2020
3. chriddyp

   changed the title [-]Incorrect docs for `sort_by`[/-] [+]Docstring generator is wrong: Incorrect docs for `sort_by`[/+] on Apr 24, 2020
4. chriddyp

   added

   bugsomething brokensomething broken

   on Apr 24, 2020
5. chriddyp commented on Apr 24, 2020

   

   chriddyp

   on Apr 24, 2020

   Member

   More actions

   While we're in the neighborhood, I'm sure there are many more fixes that we can make for markdown readability. Here are a few blocks that are pretty tough to digest:

   Missing some newlines in here? Especially to split up group and grouping
   

   JS comment included in the default text
   

   Nested dictionaries's lists aren't indendented
   

   Could use some more newlines + same nested dictionary issue
   

   null should be None
   
6. chriddyp commented on Apr 24, 2020

   

   chriddyp

   on Apr 24, 2020

   Member

   More actions

   Another related issue: https://github.com/plotly/dash-docs/issues/836
7. Marc-Andre-Rivet

   added

   size: 1

   on Apr 30, 2020
8. Marc-Andre-Rivet

   added this to the OSS milestone on Aug 4, 2020
9. chriddyp

   changed the title [-]Docstring generator is wrong: Incorrect docs for `sort_by`[/-] [+]Docstring generator is wrong for certain properties including `sort_by`[/+] on Sep 11, 2020
10. AnnMarieW

    added a commit that references this issue on Jan 17, 2021

    Update docstrings to address plotly#1205

    5e89dd4
11. alexcjohnson

    added a commit that references this issue on Feb 1, 2021

    Merge pull request #1531 from AnnMarieW/update-docstrings

    ...

    66c8451
12. HammadTheOne

    added a commit that references this issue on May 28, 2021

    Merge pull request plotly#862 from AnnMarieW/update-docstrings

    ...

    541657d
13. HammadTheOne

    added a commit that references this issue on Jul 23, 2021

    Merge pull request #862 from AnnMarieW/update-docstrings

    ...

    634f1f1
14. gvwilson commented on Jul 17, 2024

    

    gvwilson

    on Jul 17, 2024

    Contributor

    More actions

    Hi - we are tidying up stale issues and PRs in Plotly's public repositories so that we can focus on things that are most important to our community. If this issue is still a concern, please add a comment letting us know what recent version of our software you've checked it with so that I can reopen it and add it to our backlog. (Please note that we will give priority to reports that include a short reproducible example.) If you'd like to submit a PR, we'd be happy to prioritize a review, and if it's a request for tech support, please post in our community forum. Thank you - @gvwilson
15. gvwilson

    closed this as completedon Jul 17, 2024