Feature/trophies

[rolandgeider]

Proposed Changes

Shows user earned trophies

Related Issue(s)

Closes #994

[Copilot]

Pull request overview

This PR adds a trophies feature to display user-earned trophies for workout achievements. The implementation includes trophy models, a repository for fetching trophy data, UI widgets to display trophies on the dashboard and in a dedicated screen, and integration with workout logs to show personal record indicators.

Changes:

• Added trophy data models (Trophy, UserTrophy, UserTrophyProgression, ContextData) with JSON serialization
• Implemented TrophyRepository and TrophyStateNotifier using Riverpod for state management
• Created UI widgets for displaying trophies on the dashboard and in a dedicated trophies screen
• Integrated trophy indicators with workout logs to highlight personal records

Reviewed changes

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

Show a summary per file

File	Description
lib/models/trophies/*.dart	Trophy domain models with JSON serialization
lib/providers/trophies.dart	Trophy repository and state management using Riverpod
lib/widgets/trophies/*.dart	Trophy UI components for overview and dashboard
lib/widgets/dashboard/widgets/trophies.dart	Dashboard widget showing recent trophies
lib/widgets/routines/logs/*.dart	Integration of trophy/PR indicators in workout logs
lib/screens/trophy_screen.dart	Dedicated screen for viewing all trophies
lib/providers/base_provider.dart	Added language parameter support to fetch methods
lib/main.dart	Integrated Riverpod providers with existing Provider-based architecture
test/trophies/*.dart	Unit and widget tests for trophy functionality

Comments suppressed due to low confidence (2)

lib/providers/base_provider.dart:68

• The language parameter is accepted but never passed to getDefaultHeaders(). The language should be passed as getDefaultHeaders(includeAuth: true, language: language) to include the Accept-Language header in the request.

  Future<dynamic> fetch(Uri uri, {String? language}) async {
    // Send the request
    final response = await client.get(
      uri,
      headers: getDefaultHeaders(includeAuth: true),
    );

lib/widgets/routines/logs/exercises_expansion_card.dart:84

• The null assertion operator ! is used twice. Consider using a single null-safe access pattern like topSet?.weightUnitObj?.name with proper null handling to improve code clarity.

    final topSetWeightUnit = topSet?.weightUnitObj != null
        ? getServerStringTranslation(topSet!.weightUnitObj!.name, context)
        : '';

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.