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

Migrate to spatie/laravel-html #554

Closed
wants to merge 3 commits into from
Closed

Migrate to spatie/laravel-html #554

wants to merge 3 commits into from

Conversation

arborrow
Copy link
Owner

This pull request includes the changes for migrating from the laravelcollective/html package to the spatie/laravel-html package. Feel free to commit any additional changes to the shift-107204 branch.

Before merging, you need to:

  • Checkout the shift-107204 branch
  • Review all pull request comments for additional changes
  • Run composer update (if the scripts fail, add --no-scripts)
  • Clear any view cache
  • Thoroughly test your application (no tests?, no CI?)

If you need help with your migration, check out the Human Shifts.

@arborrow
Copy link
Owner Author

arborrow commented Dec 29, 2023

❌ Shift found remaining references to the Form and Html facades provided by the laravelcollective/html package. You should review these references and manually convert them to their spatie/laravel-html equivalent or raw HTML to complete your migration.

  • app/Models/Retreat.php

@arborrow
Copy link
Owner Author

⚗️ This Shift is still being refined. Please report any issues or suggestions to [email protected]. Your feedback is what helps improve the experience for everyone.

@arborrow
Copy link
Owner Author

See #557 for details about resolving Form::image

@arborrow
Copy link
Owner Author

re-running shift to test Form::image tweaks

@arborrow arborrow closed this Jan 17, 2024
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