Session-specific feedback (#110)

* Fixes a bug with a missing context because of multiple initializations of the router

* Use the session feedback API when giving feedback for a specific session

* Fix API URL construction for session feedback

* Add a testing guideline

lib/common/data/models/feedback_dto.dart

@@ -8,6 +8,7 @@ class FeedbackDTO with _$FeedbackDTO {
   factory FeedbackDTO({
     required String feedback,
     required int rating,
+    String? sessionSlug,
   }) = _FeedbackDTO;
 
   factory FeedbackDTO.fromJson(Map<String, dynamic> json) =>

lib/common/repository/api_repository.dart

@@ -140,8 +140,15 @@ class ApiRepository {
 // Submit feedback as form data
   Future<String> submitFeedback({required FeedbackDTO feedbackDTO}) async {
     try {
+      final url = StringBuffer('/events/$_eventSlug/feedback');
+
+      if (feedbackDTO.sessionSlug != null &&
+          feedbackDTO.sessionSlug!.isNotEmpty) {
+        url.write('/sessions/${feedbackDTO.sessionSlug}');
+      }
+
       final response = await _networkUtil.postReq(
-        '/events/$_eventSlug/feedback',
+        url.toString(),
         body: feedbackDTO.toJson(),
       );
 

lib/common/utils/notification_service.dart

@@ -55,8 +55,10 @@ class NotificationService {
     required String channelKey,
     required LocalSession session,
   }) async {
-    final endTime = session.endDateTime;
-    final notificationTime = endTime.subtract(const Duration(minutes: 5));
+    final notificationTime =
+        session.endDateTime.subtract(const Duration(minutes: 5));
+    // Uncomment the line below to test the notification in 5 seconds
+    // final notificationTime = DateTime.now().add(const Duration(seconds: 5));
     String? title;
     String? body;
 
@@ -73,6 +75,7 @@ class NotificationService {
           channelKey: channelKey,
           title: title,
           body: body,
+          payload: {'sessionSlug': session.slug},
         ),
         schedule: NotificationCalendar.fromDate(date: notificationTime),
       );
@@ -87,9 +90,10 @@ class NotificationService {
     switch (receivedAction.channelKey) {
       case 'session_channel':
         Logger().f('Navigating to feedback screen');
-        await getIt<FlutterConRouter>()
-            .config()
-            .push(FlutterConRouter.feedbackRoute);
+        await getIt<FlutterConRouter>().config().push(
+              FlutterConRouter.feedbackRoute,
+              extra: receivedAction.payload!['sessionSlug'],
+            );
     }
   }
 }

lib/common/utils/router.dart

@@ -81,7 +81,9 @@ class FlutterConRouter {
       GoRoute(
         path: feedbackRoute,
         name: feedbackRoute,
-        builder: (context, state) => const FeedbackScreen(),
+        builder: (context, state) => FeedbackScreen(
+          sessionSlug: state.extra as String?,
+        ),
       ),
     ],
   );

lib/features/feedback/cubit/send_feedback_cubit.dart

@@ -17,13 +17,15 @@ class SendFeedbackCubit extends Cubit<SendFeedbackState> {
   Future<void> sendFeedback({
     required String feedback,
     required int rating,
+    String? sessionSlug,
   }) async {
     try {
       emit(const SendFeedbackState.loading());
       final message = await _apiRepository.submitFeedback(
         feedbackDTO: FeedbackDTO(
           feedback: feedback,
           rating: rating,
+          sessionSlug: sessionSlug,
         ),
       );
 

lib/features/feedback/ui/feedback_screen.dart

@@ -9,7 +9,12 @@ import 'package:fluttercon/l10n/l10n.dart';
 import 'package:go_router/go_router.dart';
 
 class FeedbackScreen extends StatefulWidget {
-  const FeedbackScreen({super.key});
+  const FeedbackScreen({
+    super.key,
+    this.sessionSlug,
+  });
+
+  final String? sessionSlug;
 
   @override
   State<FeedbackScreen> createState() => _FeedbackScreenState();
@@ -151,6 +156,7 @@ class _FeedbackScreenState extends State<FeedbackScreen> {
                                       .sendFeedback(
                                         feedback: feedback,
                                         rating: selectedRating!,
+                                        sessionSlug: widget.sessionSlug,
                                       );
                                 } else {
                                   ScaffoldMessenger.of(context).showSnackBar(

pubspec.yaml

@@ -1,7 +1,7 @@
 name: fluttercon
 description: "A new Flutter project."
 publish_to: 'none'
-version: 1.13.00+11300
+version: 1.13.01+11301
 
 environment:
   sdk: ">=3.5.0 <4.0.0"