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

Installer can be used for updates #66

Merged
merged 6 commits into from
Oct 31, 2024
Merged

Installer can be used for updates #66

merged 6 commits into from
Oct 31, 2024

Conversation

woodseowl
Copy link
Collaborator

@woodseowl woodseowl commented Aug 14, 2024

Closes #63.

Allows the Starter Kit to be used as a library in an existing site, giving the option to not install all of the project and resource files and allowing for upgrading components after they are installed.

A significant portion of this update is just porting to Laravel\Prompts. There is test coverage for all of that, which has also been updated and is passing.

This PR does the following

  • Refactors starterkit:install command to utilize Laravel\Prompts
  • Updates README.md to better explain how to use installation options
  • Updates /project files to match Laravel 11 defaults (without making incompatible with L10)

To Review

  • composer require the starter kit from this branch into an existing Laravel 10 or 11 project
    composer require cornell-custom-dev/laravel-starter-kit:dev-updates
  • Install the starter kit and select the "View components" and "Example blade files" options (this will only add files to the local /resources/views directory)
    php artisan starterkit:install
  • Confirm that the contents of /vendor/cornell-custom-dev/laravel-starter-kit/resources/views has been copied to the local /resources/views directory (other than the "form" example and components, which are skipped for now).
  • Confirm that the name you gave the project shows up in the "title" attribute in /resources/views/examples/cd-index.blade.php

Copy link
Contributor

@richmarisa richmarisa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All seems fine; tested in Laravel Herd.

Copy link

@inaydich inaydich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for keeping a good documentation. I have not tested it, but I did look through the code.

@woodseowl woodseowl merged commit 68e3a8e into main Oct 31, 2024
5 checks passed
@woodseowl woodseowl deleted the updates branch October 31, 2024 18:19
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.

Ability to re-publish assets
3 participants