Update pages factory and seeder #29
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Renamed the `image` field to `feature_image` in seeders and factories for improved clarity. Updated `composer.json` to drop support for PHP 8.1, ensuring compatibility with newer versions.
Renamed `image` to `feature_image` for better clarity and added `slug` to the fillable attributes. Adjusted the `getDynamicSEOData` method to reflect these changes. This ensures alignment between database fields and application logic.
fuelviews-app bot
pushed a commit
that referenced
this pull request
Feb 12, 2025
* Refactor image field and update PHP version requirements. Renamed the `image` field to `feature_image` in seeders and factories for improved clarity. Updated `composer.json` to drop support for PHP 8.1, ensuring compatibility with newer versions. * Update image field and add slug to fillable array Renamed `image` to `feature_image` for better clarity and added `slug` to the fillable attributes. Adjusted the `getDynamicSEOData` method to reflect these changes. This ensures alignment between database fields and application logic.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull Request Description: Update Pages Factory and Seeder
This pull request updates the pages factory and seeder to ensure consistency and clarity in our data model. The primary motivation behind these changes is to replace the ambiguous 'image' field with a more descriptive 'feature_image' field across our codebase. This adjustment enhances the semantic clarity of our data model, making it clear that this field is intended for a prominent visual representation associated with the page.
Key Changes:
Composer Update: The PHP version requirement has been updated to support PHP 8.2 and above, aligning our dependencies with the latest PHP versions and ensuring compatibility with new features and improvements.
Factory Update: The
PageFactoryhas been modified to generate a 'feature_image' instead of 'image'. This aligns the generated data with our intended model structure.Seeder Update: The
PageTableSeederhas also been updated to use 'feature_image', ensuring that the seeded data is consistent with the updated factory and model.Model Update: The
Pagemodel's$fillableproperty has been updated to include 'feature_image' instead of 'image', and thegetDynamicSEODatamethod now references 'feature_image' for SEO purposes.Benefits:
This update improves the project's data handling, making it more robust and maintainable.