Skip to content

Block API users#520

Merged
michielp1807 merged 10 commits intomainfrom
block-users
Apr 1, 2026
Merged

Block API users#520
michielp1807 merged 10 commits intomainfrom
block-users

Conversation

@michielp1807
Copy link
Copy Markdown
Collaborator

@michielp1807 michielp1807 commented Mar 18, 2026

Closes #349

TODO:

  • Block users in backend
  • Add unit tests for blocked accounts
  • Update admin interface to allow blocking accounts
  • The admin interface can now also be used to delete API users (closes Delete ApiUser from admin interface #117)
  • Fix playwright tests for admin interface
  • Consider adding a "full freeze" block status to organizations

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 18, 2026

Codecov Report

❌ Patch coverage is 89.47368% with 12 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/api/oauth/handlers.rs 0.00% 5 Missing ⚠️
src/models/api_user.rs 89.18% 4 Missing ⚠️
src/api/api_users.rs 90.90% 2 Missing ⚠️
src/api/auth.rs 96.96% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@michielp1807 michielp1807 force-pushed the block-users branch 2 times, most recently from c9b232e to c578eb8 Compare March 23, 2026 15:08
@michielp1807 michielp1807 marked this pull request as ready for review March 25, 2026 11:39
@michielp1807 michielp1807 temporarily deployed to Staging UpCloud private NL-AMS March 25, 2026 11:52 — with GitHub Actions Inactive
@michielp1807 michielp1807 mentioned this pull request Mar 30, 2026
Comment thread frontend/src/routes.ts
Copy link
Copy Markdown
Collaborator

@squell squell left a comment

Choose a reason for hiding this comment

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

I've given the front-end code a more lax review (based on clicking around in staging).

@michielp1807 michielp1807 merged commit f46c8eb into main Apr 1, 2026
10 checks passed
@michielp1807 michielp1807 deleted the block-users branch April 1, 2026 07:57
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.

Make it possible to freeze/block API users Delete ApiUser from admin interface

2 participants