Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@mtaku3
Copy link
Collaborator

@mtaku3 mtaku3 commented Jan 12, 2023

This may broaden use cases.
Including,

  • 6eaa419 : fix issue showing project name instead of namespace when namespace prompt.
  • de72777 :
    • Allow any string for project name and namespace. Microsoft recommends to use PascalCase for namespace. But this is not must-do and users should be allowed to use any string as they want.
    • Allow folder name different from project name. Folder name doesn't necessarily have to be the same with project name.
  • 5a91523 :
    • fix not to singularize enum field. I think it should not be singularized. I removed that feature but can be optional.
  • d0d18bf :
    • Make singularize and pluralize of collections' name and items' name optional. According to PocketBase documentation, there's no specification to use plural for collection's name. So I made it optional (prompts at first time to generate code).

1. Allow any string for project name and namespace
   Microsoft recommends to use PascalCase for Namespace according to their styling guide. But this is not must-do.
2. Allow folder name different from project name
   Folder name doesn't necessarily have to be the same with project name.
@iluvadev
Copy link
Owner

Thanks a lot
Very good contributions! :)
I can't right now, but i will review it in a few hours

Fixes errors when there is a select type with spaces in a value.
Converts selectable values in PocketBase to PascalCase in the enum
1. Namespace with dots in words separator
2. Trimming Project and Namespace names
3. In prompt for pluralization, "Post" and "Posts" in CamelCase
4. In prompt for Directory: now the project is not created in an authomatic folder, replaced "where create the Project folder" to "where create the Project"
5. Pluralize by default:
    - When regenerates a project without this feature
    -  In prompt
@iluvadev
Copy link
Owner

Hi, I reviewed de code.
I like so much. I made some changes, minor fixes.
Thanks!

@iluvadev iluvadev merged commit 39687ab into iluvadev:main Jan 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants