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

User migration docs for 7.x -> 8.0 #3501

Merged
merged 2 commits into from
Jul 6, 2022

Conversation

vidartf
Copy link
Member

@vidartf vidartf commented Jun 23, 2022

This is meant as an explicit list of (the main) backwards incompatible changes. As this hasn't really been tracked throughout the development process, it is mainly based of the entries in the changelog. Other backwards incompatible changes probably exist, but I think we should keep this PR's review about the document structure and the items currently in it.

vidartf added 2 commits June 23, 2022 11:29
Also adds a link from the dev migration docs
to the changelog.
The plan is to release imminently, so remove this notice.
@github-actions
Copy link

Binder 👈 Launch a binder notebook on branch vidartf/ipywidgets/user-migration


Suggested migration: Rewrite all usage of `FileUpload` to use the new structure.
If you need to support both 7.x and 8.x, you can e.g. write functions `get_file_buffer` and similar
to wrap reads from the widget.
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems to me to be the largest breaking change for users. I.e. it is likely to completely break code (exceptions will be raised that cannot be recovered from), and is likely to be heavily used.

@vidartf vidartf added this to the 8.0 milestone Jun 23, 2022
@jasongrout
Copy link
Member

This looks great, thanks @vidartf!

@jasongrout jasongrout merged commit 22262e8 into jupyter-widgets:master Jul 6, 2022
@vidartf vidartf deleted the user-migration branch July 8, 2022 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants