[damianlegawiec]

Summary by CodeRabbit

• Admin Security
  • The database now supports storing failed sign-in attempt counts and account lockout timestamps for admin accounts.

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

The migration conditionally adds failed_attempts and locked_at to spree_admin_users. The schema version advances to match the migration.

Changes

Admin user lockout columns

Layer / File(s)	Summary
Add lockout columns db/migrate/20261029184031_add_lockout_columns_to_spree_admin_users.spree.rb, db/schema.rb	The migration adds failed_attempts and locked_at only when each column is absent. The schema version advances to the migration version.

Priority: Not assessed

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Other

Possibly related PRs

• spree/spree-starter#1404: Establishes the preceding schema version used by this migration series.

Suggested reviewers: mad-eel

Merge Risk: Minimal · up to 384ca

No concrete merge-blocking risk remains from this change.

Finishing Touches

📝 Generate docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

• X
• Mastodon
• Reddit
• LinkedIn

_{A rabbit checks the columns twice,
Then adds the fields with care.
The schema marks the newer date,
Its version now is there.
Soft paws hop past the migration,
And nibble carrots in the air.}

_{Comment @coderabbitai help to get the list of available commands.}