Skip to content

Comments

Multiselect diff#37

Merged
AllanKoder merged 4 commits intomasterfrom
multiselect-diff
Sep 7, 2025
Merged

Multiselect diff#37
AllanKoder merged 4 commits intomasterfrom
multiselect-diff

Conversation

@AllanKoder
Copy link
Owner

No description provided.

Copilot AI review requested due to automatic review settings September 7, 2025 00:55
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 converts the difficulty field from a single select to a multiselect implementation, allowing resources to have multiple difficulty levels.

Key changes include:

  • Database migration to change difficulty column to difficulties using MySQL SET type
  • Frontend components updated to use MultiSelect instead of Select for difficulty selection
  • Request factories and test files updated to use arrays for difficulties instead of single values

Reviewed Changes

Copilot reviewed 32 out of 32 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
Migration file Changes difficulty column to difficulties with SET type for multiple values
Request factories Updated to generate arrays for difficulties field and renamed classes for consistency
Frontend components Modified to use MultiSelect components and handle difficulty arrays
Test files Updated to use new factory class names and handle difficulties as arrays
Models/Services Added difficulties attribute accessor and updated filtering logic
Request classes Renamed for consistency and updated validation rules for difficulty arrays
Comments suppressed due to low confidence (2)

resources/js/Pages/Resources/Form/MandatoryFields.vue:1

  • Error handling should reference errors.difficulties to match the field name change.
<script setup>

resources/js/Pages/ResourceEdits/Create.vue:360

  • Validation field reference should be 'proposed_changes.difficulties' to match the updated field name.
                                            validateField(
                                                'proposed_changes.difficulty'
                                            )

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

AllanKoder and others added 2 commits September 6, 2025 19:04
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@AllanKoder AllanKoder merged commit 1b1d3ab into master Sep 7, 2025
2 of 3 checks passed
@AllanKoder AllanKoder deleted the multiselect-diff branch September 7, 2025 01:05
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