-
Notifications
You must be signed in to change notification settings - Fork 13
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
did this work on an earlier version of liveview? #2
Comments
@DarkMarmot it is likely as LiveView continues to develop, we'll need to come back to these or make a note about them being out-of-date. I've assigned @SophieDeBenedetto who is the master of all things LiveView and the creator of this repo to chime in. |
I noticed the liveview repo doesn't seem to be using tagged releases either at this point, so definitely hard to maintain as a dependency :( |
Hi @DarkMarmot, thanks for raising this issue! I am able to visit |
Hi! I checked out the master branch to see the completed project -- and maybe that was part of the problem? It looks like that branch doesn't have the seeds.exs file with demo information, so I pulled the copy from the tutorial branch and could login as Alice. I can then see the seeded chat, but if I submitted a message it would blow up. Also, the 'members' box is empty (I assume it might've had an issue?) Here's the GenServer crash on submit:
|
Doh -- i forgot to switch branches back :( Sorry -- I think the only thing I should be reporting is that you might want to copy the seeds.exs from tutorial over to the master branch. Apologies! Thanks! |
No prob! Thanks and I will definitely make that change! |
I've tried multiple branches and while I can create a user and chat rooms, there doesn't seem to be a way to view chats currently? The liveview mix dependency doesn't specify a tag to checkout -- did things break due to changes?
The text was updated successfully, but these errors were encountered: