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

fix nothing morphs and error messages #368

Merged
merged 1 commit into from
Nov 17, 2020

Conversation

leastbad
Copy link
Contributor

Type of PR (feature, enhancement, bug fix, etc.)

Bug fix

Description

Nothing morphs need to add a promise to the reflexes dictionary or else there's nothing for a non-isolated tab to run.

Also, the Log#error function was actually broken since it didn't have an internal handle to the current reflex. Not sure how that slipped past.

Fixes #363

Why should this be added

Both bugs are blockers to release.

Checklist

  • My code follows the style guidelines of this project
  • Checks (StandardRB & Prettier-Standard) are passing
  • This is not a documentation update

Please note that the best way to suggest changes or updates to the documentation is to join Discord and leave a note in the #docs channel. Any documentation updates posted as PRs cannot be accepted at this time. ❤️

@leastbad leastbad added the bug Something isn't working label Nov 16, 2020
@leastbad leastbad added this to the 3.4.0 milestone Nov 16, 2020
@leastbad leastbad self-assigned this Nov 16, 2020
@leastbad leastbad merged commit 679e342 into stimulusreflex:master Nov 17, 2020
@leastbad leastbad deleted the nothing_reflex branch November 17, 2020 04:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Console exception when reflex does not update a page that didn't trigger the reflex
1 participant