-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(template): add Adonis template #1753
feat(template): add Adonis template #1753
Conversation
This is great!! I added one comment, after adding that it's good to go! |
Ready to be merged on my side! 馃憤 |
Thank you for this! |
@allcontributors[bot] add @RomainLanz for code |
I've put up a pull request to add @RomainLanz! 馃帀 |
Adds @RomainLanz as a contributor for code. This was requested by CompuIves [in this comment](#1753 (comment))
Hey! I deployed this today, but I noticed an error with missing |
I added a PR for this: https://github.com/adonisjs/adonis-starter-codesandbox/pull/1 |
I will message him on twitter :) |
Hey! 馃憢 Should be all good by now. |
All perfect now! Thanks :D |
Hey all! 馃憢
This PR add AdonisJs as a template.
When I tested locally the code I wasn't able to load my template as server side container. It took the
create-react-app
template by default.I don't know if I need to add a
sandbox.config.json
to the root of our repository or it's normal and will be fixed when theimporters
PR will be merged?Regarding the template definition, I have not added
popular
ormain
property. I believe this can be added later if you saw it's a popular template.Importers PR: codesandbox/codesandbox-importers#36