Skip to content

add LIBRARYSYNC_MAX_USERS#44

Merged
willtho89 merged 3 commits intomainfrom
feat/user-cap
Jan 17, 2026
Merged

add LIBRARYSYNC_MAX_USERS#44
willtho89 merged 3 commits intomainfrom
feat/user-cap

Conversation

@willtho89
Copy link
Owner

No description provided.

@willtho89 willtho89 requested a review from Copilot January 17, 2026 11:49
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a configurable maximum user limit feature to librarySync. The system now enforces a cap on the number of registered users, preventing new registrations once the limit is reached.

Changes:

  • Added LIBRARYSYNC_MAX_USERS environment variable (default: 1) to control maximum allowed users
  • Implemented user count check in registration endpoint to enforce the limit
  • Updated login page template to display appropriate message when user limit is reached

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
.env.example Added LIBRARYSYNC_MAX_USERS=1 configuration variable
backend/src/librarysync/config.py Added max_users setting to Settings class with default value of 1
backend/src/librarysync/api/routes_auth.py Added user count validation in register endpoint to enforce max_users limit
backend/src/librarysync/main.py Added user count check in login route to determine if registration is full
backend/src/librarysync/templates/login.html Updated template to show "Registration full" message when user limit reached

@willtho89 willtho89 requested a review from Copilot January 17, 2026 12:04
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

@willtho89 willtho89 requested a review from Copilot January 17, 2026 12:08
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated no new comments.

@willtho89 willtho89 merged commit 5e0ccde into main Jan 17, 2026
5 checks passed
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.

1 participant