Skip to content

Remove postcodes from User Manager#2129

Merged
axlewin merged 3 commits into
mainfrom
improvement/remove-postcodes
May 1, 2026
Merged

Remove postcodes from User Manager#2129
axlewin merged 3 commits into
mainfrom
improvement/remove-postcodes

Conversation

@jsharkey13
Copy link
Copy Markdown
Member

This goes alongside isaacphysics/isaac-api#784; postcode search was unreliable and no longer fit for purpose.

This functionality is being removed from the API.
This fixes the imbalance from removing postcodes, and also puts both
the school filtering options together.
@codecov
Copy link
Copy Markdown

codecov Bot commented May 1, 2026

Codecov Report

❌ Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 43.72%. Comparing base (db9f57e) to head (a38b700).
⚠️ Report is 28 commits behind head on main.

Files with missing lines Patch % Lines
src/app/components/pages/AdminUserManager.tsx 75.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2129      +/-   ##
==========================================
- Coverage   43.72%   43.72%   -0.01%     
==========================================
  Files         584      584              
  Lines       24767    24765       -2     
  Branches     7350     8244     +894     
==========================================
- Hits        10830    10829       -1     
+ Misses      13887    13879       -8     
- Partials       50       57       +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@axlewin axlewin merged commit 208386e into main May 1, 2026
10 checks passed
@axlewin axlewin deleted the improvement/remove-postcodes branch May 1, 2026 13:56
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