Skip to content

Conversation

@FelonEkonom
Copy link
Member

No description provided.

@FelonEkonom FelonEkonom requested a review from varsill March 19, 2025 15:17
@FelonEkonom FelonEkonom self-assigned this Mar 19, 2025
@FelonEkonom FelonEkonom removed the request for review from varsill March 19, 2025 15:30
@FelonEkonom FelonEkonom requested a review from varsill March 19, 2025 16:48
Copy link
Contributor

@varsill varsill left a comment

Choose a reason for hiding this comment

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

I think there is a mismatch between the example name (live_view) and the Phoenix project name used for an actual example (example_project) which resonates throughout the READMEs etc. The best thing would be to generate a new Phoenix project with the desired project name live_view.

Another thing is that you might consider calling you example live_view_signaling instead of live_view, as it aligns more to the rest of exemplary project names

FelonEkonom and others added 13 commits March 24, 2025 12:19
Co-authored-by: Łukasz Kita <lukasz.kita0@gmail.com>
Co-authored-by: Łukasz Kita <lukasz.kita0@gmail.com>
Co-authored-by: Łukasz Kita <lukasz.kita0@gmail.com>
Co-authored-by: Łukasz Kita <lukasz.kita0@gmail.com>
Co-authored-by: Łukasz Kita <lukasz.kita0@gmail.com>
Co-authored-by: Łukasz Kita <lukasz.kita0@gmail.com>
Co-authored-by: Łukasz Kita <lukasz.kita0@gmail.com>
Co-authored-by: Łukasz Kita <lukasz.kita0@gmail.com>
Co-authored-by: Łukasz Kita <lukasz.kita0@gmail.com>
Co-authored-by: Łukasz Kita <lukasz.kita0@gmail.com>
@FelonEkonom FelonEkonom requested a review from varsill March 24, 2025 13:59
Copy link
Contributor

@varsill varsill left a comment

Choose a reason for hiding this comment

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

🥇

@FelonEkonom FelonEkonom merged commit 9c0d679 into master Mar 25, 2025
3 checks passed
@FelonEkonom FelonEkonom deleted the live-view branch March 25, 2025 10:15
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