Skip to content

Feature/fix broken routes#38

Open
buayism wants to merge 4 commits intobos-com:mainfrom
buayism:feature/fix-broken-routes
Open

Feature/fix broken routes#38
buayism wants to merge 4 commits intobos-com:mainfrom
buayism:feature/fix-broken-routes

Conversation

@buayism
Copy link
Member

@buayism buayism commented Mar 23, 2026

Fixes routing issues identified in testing:

  • Removed duplicate /health endpoint from analytics router
  • Added /dashboard endpoint placeholder for analytics
  • Added missing module docstrings to analytics, alert, and auth routers
  • Fixed /users endpoint to use proper UserCreate schema instead of OAuth2 form

buayism added 4 commits March 17, 2026 10:30
Add comprehensive .gitignore covering:
- Python bytecode and cache files
- Virtual environments
- Environment variables
- Database files
- IDE configurations
- Testing artifacts
- analytics.py: Replace duplicate /health endpoint with /dashboard
- analytics.py: Add missing module docstring
- alert_router.py: Add missing module docstring
- auth_router.py: Add missing module docstring
- auth_router.py: Fix /users endpoint to use UserCreate schema instead of OAuth2 form

Fixes routing issues where:
1. Analytics had conflicting health endpoint
2. User creation used wrong input form
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