Skip to content

Comments

Fix optional belongs_to broadcast#16

Merged
leastbad merged 2 commits intoleastbad:stream_updatesfrom
julianrubisch:fix-optional-belongs-to
Sep 22, 2021
Merged

Fix optional belongs_to broadcast#16
leastbad merged 2 commits intoleastbad:stream_updatesfrom
julianrubisch:fix-optional-belongs-to

Conversation

@julianrubisch
Copy link

@julianrubisch julianrubisch commented Sep 20, 2021

Supersedes #14 🙈

@erlingur
Copy link

For this test to fail you need to have has_many :users, broadcast: true in the Team model in the dummy app.

@julianrubisch julianrubisch changed the title WIP: Add optional belongs_to test Add optional belongs_to test Sep 22, 2021
@julianrubisch julianrubisch marked this pull request as ready for review September 22, 2021 06:44
@julianrubisch julianrubisch changed the title Add optional belongs_to test Fix optional belongs_to broadcast Sep 22, 2021
@leastbad leastbad merged commit ba81116 into leastbad:stream_updates Sep 22, 2021
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.

3 participants