Skip to content

Add support for ReactPy v2.x#54

Merged
Archmonger merged 13 commits intoreactive-python:mainfrom
shawncrawley:refactor-for-reactpy-2.0
Feb 14, 2026
Merged

Add support for ReactPy v2.x#54
Archmonger merged 13 commits intoreactive-python:mainfrom
shawncrawley:refactor-for-reactpy-2.0

Conversation

@shawncrawley
Copy link
Contributor

@shawncrawley shawncrawley commented Feb 12, 2026

Description

This is my attempt to update reactpy-router to work with the ReactPy v2.0 pre-release package.

Checklist

Please update this checklist as you complete each item:

  • Tests have been developed for bug fixes or new functionality.
  • The changelog has been updated, if necessary.
  • Documentation has been updated, if necessary.
  • GitHub Issues closed by this PR have been linked.

By submitting this pull request I agree that all contributions comply with this project's open source license(s).

@shawncrawley shawncrawley requested a review from a team as a code owner February 12, 2026 16:05
@shawncrawley
Copy link
Contributor Author

When I run the test suite locally, I still have one test failing that I couldn't figure out how to fix: test_simple_router. The failure is coming from code that was introduced years ago and hasn't changed since. It's this line here:

f"#display-{display._next_view_id}", # type: ignore

It is failing with: AttributeError: 'DisplayFixture' object has no attribute '_next_view_id'

I used various AI searches and did a lot of digging to find out how that could have ever worked and thus hopefully discover the required refactor, but unfortunately my searching came up empty.

Copy link
Contributor

@Archmonger Archmonger left a comment

Choose a reason for hiding this comment

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

I've fixed all issues within this PR, and have reviewed the changes.

Approved.

@Archmonger Archmonger changed the title Refactoring related to ReactPy v2.0.0 Add support for ReactPy v2.x Feb 14, 2026
@Archmonger Archmonger merged commit f59c85a into reactive-python:main Feb 14, 2026
9 checks passed
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