diff --git a/backend/server/adventures/views.py b/backend/server/adventures/views.py index 349c80e..f04ff1c 100644 --- a/backend/server/adventures/views.py +++ b/backend/server/adventures/views.py @@ -27,13 +27,37 @@ class AdventureViewSet(viewsets.ModelViewSet): permission_classes = [IsOwnerOrReadOnly, IsPublicReadOnly] pagination_class = StandardResultsSetPagination + def apply_sorting(self, queryset): + order_by = self.request.query_params.get('order_by', 'name') + order_direction = self.request.query_params.get('order_direction', 'asc') + + valid_order_by = ['name', 'type', 'date', 'rating'] + if order_by not in valid_order_by: + order_by = 'name' + + if order_direction not in ['asc', 'desc']: + order_direction = 'asc' + + # Apply case-insensitive sorting for the 'name' field + if order_by == 'name': + queryset = queryset.annotate(lower_name=Lower('name')) + ordering = 'lower_name' + else: + ordering = order_by + + if order_direction == 'desc': + ordering = f'-{ordering}' + + print(f"Ordering by: {ordering}") # For debugging + + return queryset.order_by(ordering) + def get_queryset(self): - lower_name = Lower('name') queryset = Adventure.objects.annotate( ).filter( Q(is_public=True) | Q(user_id=self.request.user.id) - ).order_by(lower_name) # Sort by the annotated lowercase name - return queryset + ) + return self.apply_sorting(queryset) def perform_create(self, serializer): serializer.save(user_id=self.request.user) @@ -57,20 +81,17 @@ class AdventureViewSet(viewsets.ModelViewSet): queryset |= Adventure.objects.filter( type='featured', is_public=True, trip=None) - lower_name = Lower('name') - queryset = queryset.order_by(lower_name) + queryset = self.apply_sorting(queryset) adventures = self.paginate_and_respond(queryset, request) return adventures @action(detail=False, methods=['get']) def all(self, request): - # return error if user is not authenticated if not request.user.is_authenticated: return Response({"error": "User is not authenticated"}, status=400) queryset = Adventure.objects.filter(user_id=request.user.id).exclude(type='featured') - serializer = self.get_serializer(queryset, many=True) - return Response(serializer.data) - + queryset = self.apply_sorting(queryset) + return self.paginate_and_respond(queryset, request) def paginate_and_respond(self, queryset, request): paginator = self.pagination_class() @@ -80,7 +101,6 @@ class AdventureViewSet(viewsets.ModelViewSet): return paginator.get_paginated_response(serializer.data) serializer = self.get_serializer(queryset, many=True) return Response(serializer.data) - class TripViewSet(viewsets.ModelViewSet): serializer_class = TripSerializer permission_classes = [IsOwnerOrReadOnly, IsPublicReadOnly] diff --git a/frontend/src/lib/components/Avatar.svelte b/frontend/src/lib/components/Avatar.svelte index b1d35d8..bb3833e 100644 --- a/frontend/src/lib/components/Avatar.svelte +++ b/frontend/src/lib/components/Avatar.svelte @@ -30,7 +30,7 @@

Hi, {user.first_name} {user.last_name}

  • -
  • +
  • diff --git a/frontend/src/lib/components/Navbar.svelte b/frontend/src/lib/components/Navbar.svelte index 42e36ae..f8609ff 100644 --- a/frontend/src/lib/components/Navbar.svelte +++ b/frontend/src/lib/components/Navbar.svelte @@ -58,9 +58,6 @@
  • -
  • - -
  • @@ -90,9 +87,6 @@ -
  • - -
  • diff --git a/frontend/src/routes/adventures/+page.server.ts b/frontend/src/routes/adventures/+page.server.ts index 0a0c0a9..4625918 100644 --- a/frontend/src/routes/adventures/+page.server.ts +++ b/frontend/src/routes/adventures/+page.server.ts @@ -493,7 +493,8 @@ export const actions: Actions = { adventures, next, previous, - count + count, + page } }; } catch (error) { diff --git a/frontend/src/routes/adventures/+page.svelte b/frontend/src/routes/adventures/+page.svelte index c54cd1b..d46a1cd 100644 --- a/frontend/src/routes/adventures/+page.svelte +++ b/frontend/src/routes/adventures/+page.svelte @@ -20,10 +20,13 @@ let resultsPerPage: number = 10; + let currentView: string = 'cards'; + let next: string | null = data.props.next || null; let previous: string | null = data.props.previous || null; let count = data.props.count || 0; let totalPages = Math.ceil(count / resultsPerPage); + let currentPage: number = 1; function handleChangePage() { return async ({ result }: any) => { @@ -33,6 +36,7 @@ next = result.data.body.next; previous = result.data.body.previous; count = result.data.body.count; + currentPage = result.data.body.page; totalPages = Math.ceil(count / resultsPerPage); } }; @@ -52,6 +56,7 @@ previous = result.data.previous; count = result.data.count; totalPages = Math.ceil(count / resultsPerPage); + console.log(next); } } @@ -175,31 +180,35 @@ > {sidebarOpen ? 'Close Filters' : 'Open Filters'} -
    - {#each adventures as adventure} - - {/each} -
    -
    -
    - {#if next || previous} -
    - {#each Array.from({ length: totalPages }, (_, i) => i + 1) as page} - - - - - - - {/each} -
    - {/if} + {#if currentView == 'cards'} +
    + {#each adventures as adventure} + + {/each}
    + {/if} +
    + {#if next || previous} +
    + {#each Array.from({ length: totalPages }, (_, i) => i + 1) as page} +
    + + + + {#if currentPage != page} + + {:else} + + {/if} +
    + {/each} +
    + {/if}
    @@ -261,6 +270,25 @@ on:click={() => sort({ attribute: 'name', order: 'desc' })} /> +
    +

    View

    +
    + (currentView = 'cards')} + checked + /> + (currentView = 'table')} + /> +
    diff --git a/frontend/src/routes/featured/+page.server.ts b/frontend/src/routes/featured/+page.server.ts deleted file mode 100644 index 1d3d4db..0000000 --- a/frontend/src/routes/featured/+page.server.ts +++ /dev/null @@ -1,28 +0,0 @@ -import { redirect } from '@sveltejs/kit'; -import type { PageServerLoad } from './$types'; -const PUBLIC_SERVER_URL = process.env['PUBLIC_SERVER_URL']; -import type { Adventure } from '$lib/types'; -const endpoint = PUBLIC_SERVER_URL || 'http://localhost:8000'; - -export const load = (async (event) => { - if (!event.locals.user) { - return redirect(302, '/login'); - } else { - let visitedFetch = await fetch(`${endpoint}/api/adventures/featured/`, { - headers: { - Cookie: `${event.cookies.get('auth')}` - } - }); - if (!visitedFetch.ok) { - console.error('Failed to fetch featured adventures'); - return redirect(302, '/login'); - } else { - let featured = (await visitedFetch.json()) as Adventure[]; - return { - props: { - featured - } - }; - } - } -}) satisfies PageServerLoad; diff --git a/frontend/src/routes/featured/+page.svelte b/frontend/src/routes/featured/+page.svelte deleted file mode 100644 index 3ee5473..0000000 --- a/frontend/src/routes/featured/+page.svelte +++ /dev/null @@ -1,23 +0,0 @@ - - -

    Featured Adventures

    -
    - {#each adventures as adventure} - - {/each} -
    - -{#if adventures.length === 0} -
    -

    No visited adventures yet!

    -
    -{/if}