kacerr/fuj-management
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: code quality improvements across the backend #3

Merged
kacerr merged 1 commits from claude-suggested-fixes into main 2026-03-11 10:55:52 +00:00
Conversation 0 Commits 1 Files Changed 13 +293 -88
kacerr commented 2026-03-11 10:55:45 +00:00
Owner
Copy Link
  • Remove insecure SSL verification bypass in attendance.py
  • Add gunicorn as production WSGI server (Dockerfile + entrypoint)
  • Fix silent data loss in reconciliation (log + surface unmatched members)
  • Add required column validation in payment sheet parsing
  • Add input validation on /qr route (account format, amount bounds, SPD injection)
  • Centralize configuration into scripts/config.py
  • Extract credentials path to env-configurable constant
  • Hide unmatched transactions from reconcile-juniors page
  • Fix test mocks to bypass cache layer (all 8 tests now pass reliably)
  • Add pytest + pytest-cov dev dependencies
  • Fix typo "Inffering" in infer_payments.py
  • Update CLAUDE.md to reflect current project state

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

- Remove insecure SSL verification bypass in attendance.py - Add gunicorn as production WSGI server (Dockerfile + entrypoint) - Fix silent data loss in reconciliation (log + surface unmatched members) - Add required column validation in payment sheet parsing - Add input validation on /qr route (account format, amount bounds, SPD injection) - Centralize configuration into scripts/config.py - Extract credentials path to env-configurable constant - Hide unmatched transactions from reconcile-juniors page - Fix test mocks to bypass cache layer (all 8 tests now pass reliably) - Add pytest + pytest-cov dev dependencies - Fix typo "Inffering" in infer_payments.py - Update CLAUDE.md to reflect current project state Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
kacerr added 1 commit 2026-03-11 10:55:45 +00:00
refactor: code quality improvements across the backend
All checks were successful
Deploy to K8s / deploy (push) Successful in 13s
Details
Build and Push / build (push) Successful in 32s
Details
033349cafa 
- Remove insecure SSL verification bypass in attendance.py
- Add gunicorn as production WSGI server (Dockerfile + entrypoint)
- Fix silent data loss in reconciliation (log + surface unmatched members)
- Add required column validation in payment sheet parsing
- Add input validation on /qr route (account format, amount bounds, SPD injection)
- Centralize configuration into scripts/config.py
- Extract credentials path to env-configurable constant
- Hide unmatched transactions from reconcile-juniors page
- Fix test mocks to bypass cache layer (all 8 tests now pass reliably)
- Add pytest + pytest-cov dev dependencies
- Fix typo "Inffering" in infer_payments.py
- Update CLAUDE.md to reflect current project state

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
kacerr merged commit 7d51f9ca77 into main 2026-03-11 10:55:52 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
kacerr
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: kacerr/fuj-management#3
Delete Branch "claude-suggested-fixes"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?