Skip to content

Police Auth Frontend Integration#286

Merged
naasanov merged 4 commits intomainfrom
arnav/police-auth-frontend-integration
Apr 8, 2026
Merged

Police Auth Frontend Integration#286
naasanov merged 4 commits intomainfrom
arnav/police-auth-frontend-integration

Conversation

@aLEGEND21
Copy link
Copy Markdown
Collaborator

Motivation

Due to the updated spec, it's necessary to have the ability for the application to support managing multiple police accounts.

Changes

  • Updated the police login flow so sub carries the real police account ID instead of a hardcoded string.
  • Added frontend service methods and React Query hooks for police account CRUD under /accounts/police.
  • Integrated police accounts into the admin Accounts table with edit/delete support and placeholder fields.

Closes #264

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2026

Test Results Summary

559 tests  ±0   559 ✅ ±0   37s ⏱️ ±0s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 348658b. ± Comparison against base commit 7754498.

♻️ This comment has been updated with latest results.

@naasanov naasanov mentioned this pull request Apr 7, 2026
19 tasks
* chore: Rehash police password in mock data so password verification works

* fix: hash police pwd at seed time instead of precomputed hash

* chore: fix import after pull main

---------

Co-authored-by: Nicolas Asanov <nicolas.a.asanov@gmail.com>
Co-authored-by: Nicolas Asanov <98969136+naasanov@users.noreply.github.com>
@aLEGEND21 aLEGEND21 requested a review from naasanov April 7, 2026 22:03
Copy link
Copy Markdown
Collaborator

@naasanov naasanov left a comment

Choose a reason for hiding this comment

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

lgtm

@naasanov naasanov merged commit 571c8cb into main Apr 8, 2026
3 checks passed
@naasanov naasanov deleted the arnav/police-auth-frontend-integration branch April 8, 2026 19:13
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.

Police Auth Frontend Integration

2 participants