Skip to content

Add EndlessCrashLoop LiveView to devApp#739

Merged
srzeszut merged 2 commits intomainfrom
724-handling-endless-crash-reload
Sep 11, 2025
Merged

Add EndlessCrashLoop LiveView to devApp#739
srzeszut merged 2 commits intomainfrom
724-handling-endless-crash-reload

Conversation

@srzeszut
Copy link
Contributor

No description provided.

@srzeszut srzeszut linked an issue Sep 10, 2025 that may be closed by this pull request
@srzeszut srzeszut marked this pull request as ready for review September 10, 2025 10:47
@srzeszut srzeszut changed the title Add EndlessCrashLoop live view to devApp Add EndlessCrashLoop LiveView to devApp Sep 10, 2025
Copy link
Contributor

@hhubert6 hhubert6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The reason this LiveView is currently crashing is due to lack of handle_async (you can see it in the terminal logs). I think this example would better work if you would define handle_async and raised an error inside it. That way it can be nicely seen in the debugger that this handler crashed.

Also in order for LiveDebugger to detect new LiveView is born it has to run render function. To simulate this scenario you need to give it some time, I'd add for example Process.sleep(500) inside function passed to start_async

@srzeszut srzeszut requested a review from hhubert6 September 11, 2025 09:49
@srzeszut srzeszut force-pushed the 724-handling-endless-crash-reload branch from bd3bc1c to 383877f Compare September 11, 2025 13:29
@srzeszut srzeszut merged commit 5d077d5 into main Sep 11, 2025
2 checks passed
@kraleppa kraleppa deleted the 724-handling-endless-crash-reload branch February 3, 2026 09:00
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.

Handling endless crash reload

3 participants