Skip to content

Conversation

@yCodeTech
Copy link
Owner

@yCodeTech yCodeTech commented Jan 26, 2025

Update code to reflect that of Sage v10.8.2 and require PHP 8.2

Added

  • Added a new Comments composer component in app/View/Composers/Comments.php.

Changed

  • Updated .editorconfig to use tabs instead of spaces.
  • Updated .gitignore to remove unnecessary entries.
  • Updated LICENSE.md copyright information.
  • Updated README.md badges and instructions.
  • Updated asset paths from styles and scripts to css and js in app/setup.php.
  • Updated composer.json to require PHP version 8.2 or higher.
  • Updated theme attributes in style.css and bumped PHP version.
  • Updated app/View/Composers/Post.php to add pagination functionality.

Removed

  • Removed the Alert component in app/View/Components/Alert.php.
  • Removed unnecessary components/partials.
  • Removed unnecessary items from jsconfig.
  • Removed bundles description from config/assets.php.
  • Removed soil features.
  • Removed unmaintained node-sass-glob-importer dependency.

Fixed

  • Fixed wp_link_pages pagination check to prevent rendering an empty footer element.

For more details, please refer to the pull request commits.

The `config()` method of `Application` doesn't exist in the current Acorn (v4.3.1) and this new bootloader code is not apart of Sage v10.8.2, but for an upcoming version of Sage.

Fixes PHP error: "Fatal error: Uncaught BadMethodCallException: Method Roots\Acorn\Application::configure does not exist." by reverting the code changed in commit a0d1856.
The commit a95985f removed the declaration of the `ThemeServiceProvider`. This is not needed for an upcoming version of Sage, but is needed for Sage 10.8.2.

Fixes PHP/Symfony error: "Target class [sage.view] does not exist." by re-adding the declaration of `ThemeServiceProvider` in composer.json.

Related to commit 8a9c7c2.
@yCodeTech yCodeTech merged commit 3f33826 into master Jan 27, 2025
2 of 3 checks passed
@yCodeTech yCodeTech deleted the sage-v10.8.2 branch January 27, 2025 21:56
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