Skip to content
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

Add elm to Phoenix app #52

Closed
wants to merge 12 commits into from
Closed

Add elm to Phoenix app #52

wants to merge 12 commits into from

Conversation

SimonLab
Copy link
Member

@SimonLab SimonLab self-assigned this Dec 12, 2019
@nelsonic nelsonic temporarily deployed to production-add-elm-255-pe4gcg2 December 12, 2019 14:30 Inactive
@SimonLab SimonLab added the in-progress An issue or pull request that is being worked on by the assigned person label Dec 12, 2019
@SimonLab
Copy link
Member Author

SimonLab commented Feb 5, 2020

After reviewing this PR, I don't think it worth merging it on the mvp repo.
I've opened it at the begining of the spike: dwyl/app#254 however since then another repository explained how to create an Elm PWA application: https://github.com/dwyl/elm-pwa-example

I've also updated the spike issue with my latest thought and conclude that creating a PWA with Elm and Phoenix is not obvious and might be the right way to go see: dwyl/app#254 (comment)

Also from the issue dwyl/app#266 having PWA is not an MVP feature.

@SimonLab SimonLab closed this Feb 5, 2020
@SimonLab SimonLab deleted the add-elm-#255 branch February 5, 2020 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in-progress An issue or pull request that is being worked on by the assigned person
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants