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

Release 8.13.0 - Laravel 11 upgrade #746

Merged
merged 22 commits into from
Nov 6, 2024
Merged

Release 8.13.0 - Laravel 11 upgrade #746

merged 22 commits into from
Nov 6, 2024

Conversation

nickdenardis
Copy link
Member

Dependency updates

Updated codebase to Laravel 11.

We used Laravel Shift to update the code automatically and it supplied us with changes it could not do on its own. This process saved at least 10 hours of developer time.

The result is a slimmer codebase (all default configuration files have been removed) the defaults are now built into the framework codebase and only changes need to be added to the application code.

Transition

With this change, it means more items are defined directly in the .env file. To upgrade sites, we will be rolling out new .env updates before upgrading the site to ensure our unique configuration values are set on deployment.

Features

There are no user-facing feature updates in this release.

@nickdenardis nickdenardis added the dependencies Pull requests that update a dependency file label Nov 6, 2024
@nickdenardis nickdenardis added this to the 8.13.0 Release milestone Nov 6, 2024
@nickdenardis nickdenardis self-assigned this Nov 6, 2024
@nickdenardis nickdenardis requested a review from a team as a code owner November 6, 2024 11:44
@coveralls
Copy link

Coverage Status

coverage: 100.0%. remained the same
when pulling 8ac20f1 on release/8.13.0
into c83e07b on master.

@AmbyrElan
Copy link
Contributor

@nickdenardis ,

In reference to the dependency updates where '(all default configuration files have been removed) the defaults are now built into the framework codebase and only changes need to be added to the application code,' does this effect how we assign config items like controllers and global site promotions? Would we no longer use the config.base file to handle those assignments and handle those adjustments in the template controller instead?

@nickdenardis nickdenardis merged commit e1c0288 into master Nov 6, 2024
2 checks passed
@nickdenardis nickdenardis deleted the release/8.13.0 branch November 6, 2024 18:11
@nickdenardis
Copy link
Member Author

@nickdenardis ,

In reference to the dependency updates where '(all default configuration files have been removed) the defaults are now built into the framework codebase and only changes need to be added to the application code,' does this effect how we assign config items like controllers and global site promotions? Would we no longer use the config.base file to handle those assignments and handle those adjustments in the template controller instead?

@AmbyrElan It does not affect our code; it only affects framework-level configurations.

The base.php file has not been modified in this upgrade:
https://github.com/waynestate/base-site/blob/master/config/base.php

@AmbyrElan
Copy link
Contributor

@nickdenardis ,

In reference to the dependency updates where '(all default configuration files have been removed) the defaults are now built into the framework codebase and only changes need to be added to the application code,' does this effect how we assign config items like controllers and global site promotions? Would we no longer use the config.base file to handle those assignments and handle those adjustments in the template controller instead?

@AmbyrElan It does not affect our code; it only affects framework-level configurations.

The base.php file has not been modified in this upgrade:
https://github.com/waynestate/base-site/blob/master/config/base.php

Thanks for clarifying!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants