-
-
Notifications
You must be signed in to change notification settings - Fork 708
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
Bump stimulus_reflex to 3.5.0-rc4 and cable_ready to 5.0.3 #12235
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
2276b55
to
a3c239a
Compare
Yay, after upgrading stimulus_reflex, this now works! |
a3c239a
to
c4df318
Compare
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.
👍
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.
Great! 🤞
Does this need testing? Maybe checking reports? |
Yes, I think it should be treated the same as #12260 , which is awaiting testing, but also lacking test notes. |
Bumps [stimulus_reflex](https://github.com/stimulusreflex/stimulus_reflex) from 3.5.0-rc3 to 3.5.0-rc4. - [Release notes](https://github.com/stimulusreflex/stimulus_reflex/releases) - [Changelog](https://github.com/stimulusreflex/stimulus_reflex/blob/main/CHANGELOG.md) - [Commits](https://github.com/stimulusreflex/stimulus_reflex/commits) --- updated-dependencies: - dependency-name: stimulus_reflex dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
Gem and package need to be in sync.
Unfortunately the testing framework doesn't support the latest version of StimulusReflex. This is a third party framework that doesn't seem to be maintained. Maybe we should convert these to ActionCable tests.
It contains a fix for stimulus_reflex 3.5.0-rc4, but hasn't been released yet. podia/stimulus_reflex_testing#21 Now the user reflex spec works, but the products spec needs another fix. But we plan to remove it soon anyway.
In order to match installed js package version. For some reason, it still reports js package as 5.0.1..
I don't know why cable_ready@^5.0.0 would resolve to v5.0.2, when v5.0.3 is available. So I gave yarn a bit of a nudge in the right direction. It installs fine now.
c4df318
to
b7092a5
Compare
Oh, yes thanks. That's weird, I don't know how I missed this. |
main
(because latest fixes haven't been released yet).What should we test?
These are underlying frameworks used for the dynamic functionality on the below pages. Simply checking that they work as usual, with no errors in the dev console should be sufficient.
/admin/products
withadmin_style_v3
)/admin/reports/packing/customer
(old or new design, doesn't matter)Dependencies