Skip to content
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

Improve deprecation docs for Node.from_parent #6294

Closed
4 tasks done
nicoddemus opened this issue Nov 30, 2019 · 0 comments · Fixed by #6680
Closed
4 tasks done

Improve deprecation docs for Node.from_parent #6294

nicoddemus opened this issue Nov 30, 2019 · 0 comments · Fixed by #6680
Assignees
Labels
status: critical grave problem or usability issue that affects lots of users type: deprecation feature that will be removed in the future type: docs documentation improvement, missing or needing clarification
Milestone

Comments

@nicoddemus
Copy link
Member

nicoddemus commented Nov 30, 2019

In the "Node Construction changed to Node.from_parent" section in the deprecation docs, we definitely need to add:

  • An example of the warning that users will see (so they can find the session on google).
  • The warning NODE_USE_FROM_PARENT should point to the deprecation docs.
  • Show a "before -> after" example.
  • ensure from_parent will not support config/session
@nicoddemus nicoddemus added type: docs documentation improvement, missing or needing clarification type: deprecation feature that will be removed in the future labels Nov 30, 2019
@nicoddemus nicoddemus added this to the 5.4 milestone Nov 30, 2019
@RonnyPfannschmidt RonnyPfannschmidt added the status: critical grave problem or usability issue that affects lots of users label Jan 15, 2020
@RonnyPfannschmidt RonnyPfannschmidt moved this from To do to Done in detangle node structure Aug 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: critical grave problem or usability issue that affects lots of users type: deprecation feature that will be removed in the future type: docs documentation improvement, missing or needing clarification
Projects
Development

Successfully merging a pull request may close this issue.

2 participants