Skip to content

Remove the Admin Seeder and Update the CONTRIBUTING.md#758

Merged
allella merged 3 commits into
hackgvl:developfrom
allella:remove-user-admin-migration
Jul 16, 2026
Merged

Remove the Admin Seeder and Update the CONTRIBUTING.md#758
allella merged 3 commits into
hackgvl:developfrom
allella:remove-user-admin-migration

Conversation

@allella

@allella allella commented Jul 15, 2026

Copy link
Copy Markdown
Member
  • Removes a user seeder that was creating a default users that's not needed. It was a bit of a liability to create it because anybody could see the default user info in the public seeder file.
  • Add Filament command examples to the CONTRIBUTING.md
  • Added a note to CONTRIBUTING.md explaining that, by default, Laravel tests run independent of database seeders, but you can explicitly enable them. There was confusion in Tests: Latest HomeController Tests are Failing #750 (comment) about if seeders were visible to the tests.

@allella
allella requested a review from bogdankharchenko July 15, 2026 14:25
@allella
allella merged commit b5a0fee into hackgvl:develop Jul 16, 2026
1 check passed
@allella
allella deleted the remove-user-admin-migration branch July 16, 2026 03:08
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