-
-
Notifications
You must be signed in to change notification settings - Fork 40
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
Enable Rails Controller Generator to create Phlex Views instead of ERB templates #208
Comments
Sounds great! Give me a shout if you have any questions. |
Hey @rubyforbusiness did you get anywhere with this? I'm pretty much betting the farm on Phlex (as in, all the apps (a relatively large amount) I'm creating from now on are going using Phlex), so I'm really keen on this (and other things, but this would be awesome ASAP). Let me know if you'd like to discuss etc. |
Hi @trinitytakei , yes I'm currently working on this, a little sporadically as I'm also looking for paid employment! But I'm really excited about the potential of Phlex and like you, already introduced it into a production system I was working on early this year. I plan to do some more work on it this coming week. Here's where I've got to so far:
Obviously I'm making up some conventions regarding the naming of things, but we can discuss that at PR time. What I'm excited to do in future tickets, and never got a chance on the production project I mentioned, is to really start taking advantage of an OOUI to do things like:
|
count me in! |
Thanks for assigning this to me @joeldrapper . I've done the code and am in the process of writing some tests. |
Sorry for the delay in doing this work. I've been working on a short-term contract which is the first paid work I've had in quite a while. Now I'm looking for another job. I was going to finish this ticket today, but when I tried to rebase to the latest main, I realised there have been quite a few changes to the |
I've just had a thought. I bet a lot of people are still on Ruby 2 and can't upgrade that easily yet would like to try Phlex. I could put my current change into a patch release for |
Hi @joeldrapper I've created a PR for the v1 changes: #234. |
FYI @joeldrapper , I'm working on the v2 changes in a separate branch. Whilst I'm there I'm going to change the existing tests to use the new namespacing e.g. get rid of |
Hi @joeldrapper , I've created another PR for the v2 changes: #237 |
@trinitytakei , @julianrubisch I'd be interested in your input on this, now there's something to try :-). |
I guess an obvious improvement, if these PRs go in, is to add the 'scaffold' examples to the handbook on phlex.fun and beta.phlex.fun. |
Back in 2022 @joeldrapper [said] (https://github.com/orgs/phlex-ruby/discussions/235#discussioncomment-3852112):
I'd like to have a go at implementing this as it would really enable people to see the power of OO views quickly, easily and from a familiar starting point.
The text was updated successfully, but these errors were encountered: