Skip to content

fix id for pgsql #1748

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
dbarzin merged 1 commit into from
Nov 20, 2025
Merged

fix id for pgsql #1748

dbarzin merged 1 commit into from
Nov 20, 2025

Conversation

@dbarzin
Copy link
Owner

@dbarzin dbarzin commented Nov 20, 2025 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

  • Chores
    • Modified database initialization to use auto-generated identifiers for user records during seeding, streamlining the seed data configuration.

✏️ Tip: You can customize this high-level summary in your review settings.

@dbarzin dbarzin merged commit 95f1715 into master Nov 20, 2025
4 of 5 checks passed
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 20, 2025 •
edited
Loading

Caution

Review failed

The pull request is closed.

Walkthrough

Removed the explicit id => 1 field assignment from the admin user seed record in the UsersTableSeeder. The user creation now relies on the database's auto-increment mechanism to assign the ID value automatically.

Changes

Cohort / File(s) Change Summary
Seed Data Modification
database/seeders/UsersTableSeeder.php		 Removed explicit id => 1 assignment from the first user entry; database auto-increment will now handle ID assignment

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 One line hops away,
No ID to say,
Let the database play,
Auto-increment's way! 🌱

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch dev
📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Cache: Disabled due to data retention organization setting

Knowledge base: Disabled due to data retention organization setting

📥 Commits

Reviewing files that changed from the base of the PR and between 5d17496 and 9b4e264.

📒 Files selected for processing (1)
  • database/seeders/UsersTableSeeder.php (0 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dbarzin