Skip to content

feat: add shield:publish CLI Command - #489

Draft
datamweb wants to merge 2 commits into
codeigniter4:developfrom
datamweb:add-publish-CLIcommand
Draft

feat: add shield:publish CLI Command#489
datamweb wants to merge 2 commits into
codeigniter4:developfrom
datamweb:add-publish-CLIcommand

Conversation

@datamweb

Copy link
Copy Markdown
Collaborator

Hello friends,
This PR helps to create custom views and adjust it. (related to: #480 )
It also helps to create custom UserModel. (related to: #476)

  • php spark shield:publish view
  • php spark shield:publish model

I'm not sure if merging this command is really necessary, however if you think it might be useful please give it a first look.

@datamwebdatamweb added enhancement New feature or request tests needed Pull requests that need tests docs needed Pull requests needing documentation write-ups and/or revisions. labels Oct 30, 2022
@kenjis

Copy link
Copy Markdown
Member

@datamweb

datamweb commented Oct 30, 2022

Copy link
Copy Markdown
CollaboratorAuthor

Kenjis, you are right. I don't agree with the name shield:publish either, to be honest, I already chose shield:custom.
However, shield:make is good, but you have to be careful, in make:... , the adjustment work is never done, here after the copy views, the adjustment of the views is done.

I was going to use GeneratorTrait to write command shield:publish model, but GeneratorTrait didn't have the necessary flexibility, it seems that some changes must be made in GeneratorTrait , which I don't intend to go into.

@kenjis

Copy link
Copy Markdown
Member

Ah, the view command is a bit different from make command.
make generates files from templates.
But the view command copies view files and fixes the Config file.

@kenjis

Copy link
Copy Markdown
Member

This is the output of Myth Auth auth:publish command.
It seems the view command is the same as the part of Publish Views?.

$ php spark auth:publishCodeIgniter v4.1.5 Command Line Tool - Server Time: 2021-12-09 16:36:40 UTC+09:00Publish Migration? [y, n]: y Created: Database/Migrations/2017-11-20-223112_create_auth_tables.php Remember to run `spark migrate -all` to migrate the database.Publish Models? [y, n]: y Created: Models/LoginModel.php Created: Models/UserModel.phpPublish Entities? [y, n]: y Created: Entities/User.phpPublish Controller? [y, n]: y Created: Controllers/AuthController.phpPublish Views? [y, n]: y Created: Views/Auth/_footer.php Created: Views/Auth/_header.php Created: Views/Auth/_message_block.php Created: Views/Auth/_navbar.php Created: Views/Auth/emails/activation.php Created: Views/Auth/emails/forgot.php Created: Views/Auth/forgot.php Created: Views/Auth/layout.php Created: Views/Auth/login.php Created: Views/Auth/register.php Created: Views/Auth/reset.phpPublish Filters? [y, n]: y Created: Filters/LoginFilter.php Created: Filters/PermissionFilter.php Created: Filters/RoleFilter.phpPublish Config file? [y, n]: y Created: Config/Auth.phpPublish Language file? [y, n]: y Created: Language/en/Auth.php

@datamweb

Copy link
Copy Markdown
CollaboratorAuthor

But the view command copies view files and fixes the Config file.

Exactly.

make generates files from templates.

Yes, if you want I can write it with GeneratorTrait.

@datamweb

Copy link
Copy Markdown
CollaboratorAuthor

It seems the view command is the same as the part of Publish Views?.

I haven't used auth:publish before, but that's what I see.
I don't think we need to cover all of this, some of it is in shield:setup. However, I want to see what you mean?
If you think there is a need for changes or something should be added, I will wait for you.

@kenjis

Copy link
Copy Markdown
Member

I think the following commands are better:

  • php spark shield:publish view
  • php spark shield:make model

@kenjis

Copy link
Copy Markdown
Member

shield:model was implemented in #558

@datamweb

datamweb commented Oct 19, 2023

Copy link
Copy Markdown
CollaboratorAuthor

@kenjis We have now documented the customization details for views.
I want to know we need php spark shield:publish view now?

@kenjis

Copy link
Copy Markdown
Member

It is assumed that all view files will be customized by developers.

Because of the large number of files, it would be convenient if they can be prepared by one command.
So I would be happy to have it.
However, I don't mind if it is not in v1.0.0.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs neededPull requests needing documentation write-ups and/or revisions.enhancementNew feature or requesttests neededPull requests that need tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@datamweb@kenjis