Skip to content

complain loudly if LV is initialized twice#3721

Closed
SteffenDE wants to merge 2 commits intomainfrom
sd-duplicate-init
Closed

complain loudly if LV is initialized twice#3721
SteffenDE wants to merge 2 commits intomainfrom
sd-duplicate-init

Conversation

@SteffenDE
Copy link
Collaborator

also warn if view is bound to an already bound element

See: https://elixirforum.com/t/how-do-you-debug-state-changes-using-liveviews/70044/21

@SteffenDE
Copy link
Collaborator Author

image

@SteffenDE SteffenDE marked this pull request as ready for review March 26, 2025 11:26
@SteffenDE SteffenDE added this to the v1.1 milestone Mar 26, 2025
@josevalim
Copy link
Member

Do we need this now that we got rid of nested layouts by default? Probably not worth the check anymore?

@SteffenDE
Copy link
Collaborator Author

I still think it doesn't hurt

SteffenDE added a commit that referenced this pull request May 20, 2025
@SteffenDE SteffenDE closed this May 20, 2025
@SteffenDE SteffenDE deleted the sd-duplicate-init branch May 20, 2025 14:49
SteffenDE added a commit that referenced this pull request May 20, 2025
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.

2 participants