[FIX]: Users demo

[snewcomer]

close #43 #51

[jg-made]

@snewcomer I might be doing something wrong locally but when I run your branch and try to delete a user I get an error message:
value `%{"altKey" => false, "ctrlKey" => false, "metaKey" => false, "pageX" => 1255, "pageY" => 180, "screenX" => 1255, "screenY" => 309, "shiftKey" => false, "x" => 1255 , "y" => 180}` in `where` cannot be cast to type :id in query:
It looks like the link being used to delete a user in the index.html template is not sending the user id but instead sending details about the click event.

Thanks for the fork though! I am trying to write a generic master-detail view where the master list updates when a new "thing" is created or when a "thing" is deleted.

[snewcomer]

@jg-made great find 👍 I fixed and added a test

[jg-made]

@snewcomer thanks again! Great work on making it compatible with modern live view.

[jg-made]

@snewcomer I have another question please. I see that in user_live/index.ex we have a handle_info/2 function which looks like it is written to handle all kinds of :user changes (:updated, :deleted, etc.).
However, if I have two separate pages open, say the first one on the index list view and say in the second page I delete or update or add a user, the list on the first page does not update. I think maybe the index view has not subscribed to the user socket properly?

UPDATE: OMG i actually figured it out. You need to add this line:

if connected?(socket), do: Demo.Accounts.subscribe()

to the handle_params/3 function in user_live/index.ex.

[jg-made]

add this line

if connected?(socket), do: Demo.Accounts.subscribe()

[snewcomer]

Thank you thank you! Updated and nice job 👍

[snewcomer]

too many merge conflicts