onSubscribed reads best

Since it's a callback after the purchase completes. The user will be already subscribed.
canonical · Sep 6, 2023 · c5cabcc · c5cabcc
1 parent 6ce676b
commit c5cabcc
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/gui/packages/ubuntupro/lib/pages/subscription_status/subscribe_now_page.dart b/gui/packages/ubuntupro/lib/pages/subscription_status/subscribe_now_page.dart
@@ -9,8 +9,8 @@ import 'subscribe_now_widgets.dart';
 import 'subscription_status_model.dart';
 
 class SubscribeNowPage extends StatelessWidget {
-  const SubscribeNowPage({super.key, required this.onSubscribe});
-  final void Function(SubscriptionInfo) onSubscribe;
+  const SubscribeNowPage({super.key, required this.onSubscribed});
+  final void Function(SubscriptionInfo) onSubscribed;
 
   @override
   Widget build(BuildContext context) {
@@ -43,7 +43,7 @@ class SubscribeNowPage extends StatelessWidget {
                       ),
                     );
                   },
-                  ifRight: onSubscribe,
+                  ifRight: onSubscribed,
                 );
               },
               child: Text(lang.subscribeNow),

diff --git a/gui/packages/ubuntupro/lib/pages/subscription_status/subscription_status_page.dart b/gui/packages/ubuntupro/lib/pages/subscription_status/subscription_status_page.dart
@@ -47,7 +47,7 @@ class SubscriptionStatusPage extends StatelessWidget {
             caption: lang.orgManaged,
           ),
         SubscribeNowModel() => SubscribeNowPage(
-            onSubscribe: (info) =>
+            onSubscribed: (info) =>
                 context.read<ValueNotifier<SubscriptionInfo>>().value = info,
           ),
       },

diff --git a/gui/packages/ubuntupro/test/pages/subscription_status/subscribe_now_page_test.dart b/gui/packages/ubuntupro/test/pages/subscription_status/subscribe_now_page_test.dart
@@ -119,7 +119,7 @@ Widget buildApp(
         body: Provider.value(
           value: model,
           child: SubscribeNowPage(
-            onSubscribe: onSubs,
+            onSubscribed: onSubs,
           ),
         ),
       ),