Consolidate API root to /api/ path, remove duplicate endpoint at /

octofit-tracker/backend/octofit_tracker/urls.py

@@ -13,48 +13,24 @@
     1. Import the include() function: from django.urls import include, path
     2. Add a URL to urlpatterns:  path('blog/', include('blog.urls'))
 """
-import os
 from django.contrib import admin
 from django.urls import path, include
 from rest_framework import routers
-from rest_framework.decorators import api_view
-from rest_framework.response import Response
 from octofit_tracker import views
 
 # Create a router and register viewsets
-router = routers.DefaultRouter()
+# Using SimpleRouter since we have a custom api_root view
+router = routers.SimpleRouter()
 router.register(r'users', views.UserViewSet)
 router.register(r'teams', views.TeamViewSet)
 router.register(r'activities', views.ActivityViewSet)
 router.register(r'leaderboard', views.LeaderboardViewSet)
 router.register(r'workouts', views.WorkoutViewSet)
 
-
-@api_view(['GET'])
-def api_root_codespace(request, format=None):
-    """
-    API root endpoint that returns full URLs for codespace environment.
-    """
-    codespace_name = os.environ.get('CODESPACE_NAME')
-
-    if codespace_name:
-        # Running in GitHub Codespaces
-        base_url = f'https://{codespace_name}-8000.app.github.dev'
-    else:
-        # Running locally
-        base_url = f'{request.scheme}://{request.get_host()}'
-
-    return Response({
-        'users': f'{base_url}/api/users/',
-        'teams': f'{base_url}/api/teams/',
-        'activities': f'{base_url}/api/activities/',
-        'leaderboard': f'{base_url}/api/leaderboard/',
-        'workouts': f'{base_url}/api/workouts/',
-    })
-
-
 urlpatterns = [
-    path('', api_root_codespace, name='api-root'),
-    path('api/', include(router.urls)),
     path('admin/', admin.site.urls),
+    # API root view - must come before router include to match /api/ exactly
+    path('api/', views.api_root, name='api-root'),
+    # Router URLs handle /api/users/, /api/teams/, etc.
+    path('api/', include(router.urls)),
 ]