Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Unable to load builder with default generator output #137

Closed
coder2000 opened this issue Feb 7, 2023 · 1 comment · Fixed by #138
Closed

Unable to load builder with default generator output #137

coder2000 opened this issue Feb 7, 2023 · 1 comment · Fixed by #138
Assignees
Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request
Milestone

Comments

@coder2000
Copy link

The default location that the generator places the builder class of lib/ isn't loaded automatically by the application and needs to be manually included somewhere. There are are few options:

  1. Update the documentation to specify that it needs to be included
  2. Add an initializer to the generator that requires the builder
  3. Change the generator to put the builder in the app/helpers folder instead
@Spone Spone added bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request labels Feb 7, 2023
@Spone Spone added this to the v0.3 milestone Feb 7, 2023
@Spone Spone self-assigned this Feb 7, 2023
@Spone
Copy link
Collaborator

Spone commented Feb 9, 2023

I'd be OK with option 1, option 3 (but maybe with an app/forms folder?), or suggesting both 1 and 3 in the docs.
What do you think @nicolas-brousse?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants