Skip to content
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

Improve the artisan commands #1293

Merged
merged 7 commits into from
Jul 5, 2024

Conversation

dfsmania
Copy link
Collaborator

@dfsmania dfsmania commented Jul 3, 2024

Question Answer
Pull request type ENHANCEMENT
License MIT

What's in this PR?

Improve the artisan commands available on this package:

AdminLteInstallCommand

  • basic_routes resources renamed to auth_routes.
  • Installation of type enhanced was removed, and replaced by basic_with_auth (basic installation + resources related to Laravel/UI) and basic_with_views (basic installation + layouts views).
  • The installation of type full now publish all the package resources.

AdminLteStatusCommand

  • Logic of the command was improved.
  • The resources status table now shows a new column named Publishing Target with the installation target folder or file for each resource.
  • The style of the Required column was improved.

AdminLtePluginsCommand

  • Logic of the command was improved.

Checklist

  • I tested these changes.

@dfsmania dfsmania marked this pull request as draft July 3, 2024 23:15
@dfsmania dfsmania marked this pull request as ready for review July 5, 2024 14:14
@dfsmania dfsmania requested a review from resslinger July 5, 2024 14:15
@resslinger resslinger merged commit 81a39e5 into jeroennoten:master Jul 5, 2024
34 checks passed
@dfsmania dfsmania deleted the improve_artisan_commands branch July 5, 2024 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants