Skip to content

Conversation

@SebKay
Copy link
Owner

@SebKay SebKay commented Dec 8, 2025

Note

Introduce a shared Enum trait for value access, apply it to all enums, update migrations/UI to use it, and hash seeded user passwords in the factory.

  • Enums:
    • Add App\Enums\Concerns\Enum trait with values() and only() helpers.
    • Apply trait to Environment, Permission, Queue, and Role; remove per-enum value helpers.
  • Database:
    • Update permission/role enum columns to use Permission::only([...], true)->all() and Role::only([...], true)->all().
  • UI:
    • Use Role::values()->all() for the Roles filter options in app/Filament/Resources/Users/Tables/UsersTable.php.
  • Factories:
    • Hash seeded passwords via Hash::make and pull credentials from config() in database/factories/UserFactory.php.

Written by Cursor Bugbot for commit 8b2774b. This will update automatically on new commits. Configure here.

@SebKay SebKay self-assigned this Dec 8, 2025
@SebKay
Copy link
Owner Author

SebKay commented Dec 8, 2025

@cursor review

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no bugs!


Fixes issues where you add new enum values later but migrations have already been run
@SebKay
Copy link
Owner Author

SebKay commented Dec 8, 2025

@cursor review

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no bugs!


@SebKay SebKay merged commit 1ad80bb into main Dec 8, 2025
2 checks passed
@SebKay SebKay deleted the enum-trait branch December 8, 2025 07:31
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