Skip to content

Comments

Tag Changes#43

Merged
AllanKoder merged 2 commits intomasterfrom
more-bugs-and-fixes
Oct 7, 2025
Merged

Tag Changes#43
AllanKoder merged 2 commits intomasterfrom
more-bugs-and-fixes

Conversation

@AllanKoder
Copy link
Owner

No description provided.

Copilot AI review requested due to automatic review settings October 7, 2025 16:09
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR renames tag-related fields from singular to plural forms throughout the codebase. The changes standardize field naming conventions by updating topic_tags to topics_tags and programming_language_tags to programming_languages_tags.

  • Renamed tag fields to use consistent plural naming conventions
  • Updated all references across models, controllers, views, tests, and database factories
  • Added a new console command for updating tag frequencies

Reviewed Changes

Copilot reviewed 23 out of 23 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/RequestFactories/StoreResourceRequestFactory.php Updated factory to use new plural tag field names
tests/RequestFactories/StoreResourceEditRequestFactory.php Updated factory and comment to use new plural tag field names
tests/Feature/TagSearchTest.php Updated test data to use new plural tag field names
tests/Feature/ResourceEditsTest.php Updated test assertions and data to use new plural tag field names
tests/Feature/ComputerScienceResourceTest.php Updated test data provider to use new plural tag field names
resources/js/Pages/Resources/Form/TagsFields.vue Updated Vue component bindings to use new plural tag field names
resources/js/Pages/Resources/Create.vue Updated form data structure to use new plural tag field names
resources/js/Pages/ResourceEdits/Show.vue Updated field configuration to use new plural tag field names
resources/js/Pages/ResourceEdits/Create.vue Updated form data and validation to use new plural tag field names
resources/js/Components/Resources/ResourceItem.vue Updated template references to use new plural tag field names
resources/js/Components/Resources/ResourceDescription.vue Updated template references to use new plural tag field names
database/factories/ResourceEditsFactory.php Updated factory to use new plural tag field names
database/factories/ComputerScienceResourceFactory.php Updated factory to use new plural tag field names
app/Services/ComputerScienceResourceService.php Updated service to use new plural tag field names
app/Observers/ComputerScienceResourceObserver.php Updated observer to use new plural tag field names
app/Models/ResourceEdits.php Updated slug configuration to include resource ID and ensure uniqueness
app/Models/ComputerScienceResource.php Updated model properties, accessors, and guarded fields to use new plural tag field names
app/Http/Resources/ComputerScienceResourceResource.php Updated API resource to use new plural tag field names
app/Http/Requests/StoreResourceRequest.php Updated validation rules to use new plural tag field names
app/Http/Requests/StoreResourceEditRequest.php Updated validation rules to use new plural tag field names
app/Http/Controllers/ResourceEditsController.php Updated controller to use new plural tag field names
app/Filament/Admin/Resources/ComputerScienceResource.php Updated admin interface to use new plural tag field names
app/Console/Commands/UpdateTagFrequencies.php Added new console command for recalculating tag frequencies

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@AllanKoder AllanKoder merged commit da6b7fd into master Oct 7, 2025
1 check passed
@AllanKoder AllanKoder deleted the more-bugs-and-fixes branch October 7, 2025 16:14
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.

1 participant