Merge pull request #40 from Iconica-Development/fix/splashscreen-future

fix: always call the splashHandler with the splashScreenFuture and killswitchService when a splashScreenBuilder is provided

.github/workflows/release.yml

@@ -0,0 +1,14 @@
+name: Iconica Standard Component Release Workflow
+# Workflow Caller version: 1.0.0
+
+on:
+  release:
+    types: [published]
+    
+  workflow_dispatch:
+
+jobs:
+  call-global-iconica-workflow:
+    uses: Iconica-Development/.github/.github/workflows/component-release.yml@master
+    secrets: inherit
+    permissions: write-all

.gitignore

@@ -44,3 +44,7 @@ app.*.map.json
 ios
 .metadata
 pubspec.lock
+
+# FVM Version Cache
+.fvm/
+.fvmrc

CHANGELOG.md

@@ -1,3 +1,20 @@
+## 4.2.4
+- Fixed the userstory to always call the splashScreenFuture and killswitchservice logic when a custom splashScreenBuilder is provided
+
+## 4.2.3
+- Added check if introduction should be shown according to the service before showing the introduction at all
+
+## 4.2.2
+- Added custom navigator in the root of the navigator user-story
+
+## 4.2.1
+- Updated flutter_introduction to 5.0.0
+
+## 4.1.0
+- Updated README
+- Removed check if the introductions should be shown.
+- Updated flutter_introduction to 3.1.0
+
 ## 4.0.0
 - Added default introduction page.
 - Added default splash screen.

README.md

@@ -6,60 +6,65 @@ Flutter_start is a package that allows you to jumpstart your application with a
 
 To use this package, add flutter_start as a dependency in your pubspec.yaml file:
 
-```
+```yaml
   flutter_start:
     git:
       url: https://github.com/Iconica-Development/flutter_start
-      ref: <Version>
+      ref: 4.1.0
 ```
 
-To use the module within your Flutter-application with predefined `Go_router` routes you should add the following:
+## go_router
+
+Add `go_router` as dependency to your project.
 
-Add go_router as dependency to your project.
 Add the following configuration to your flutter_application:
 
 ```
-StartUserStoryConfiguration startUserStoryConfiguration = const StartUserStoryConfiguration();
+List<GoRoute> getStartStoryRoutes() => getStartStoryRoutes();
 ```
 
-and set the values as you wish.
+Or with options:
 
-Next add the `StartUserStoryConfiguration` to `getStartStoryRoutes` Like so:
+Place the following code somewhere in your project where it can be accessed globally:
 
 ```
-List<GoRoute> getStartRoutes() => getStartStoryRoutes(
+var startUserStoryConfiguration = const StartUserStoryConfiguration();
-      startUserStoryConfiguration,
+```
+
+```
+List<GoRoute> getStartStoryRoutes() => getStartStoryRoutes(
+      configuration: startUserStoryConfiguration,
     );
 ```
 
-Finally add the `getStartRoutes` to your `Go_router` routes like so:
+Finally add the `getStartRoutes` to your `go_router` routes like so:
 
 ```
 final GoRouter _router = GoRouter(
   routes: <RouteBase>[
-    ...getStartRoutes()
+    ...getStartStoryRoutes()
   ],
 );
 ```
 
 The routes that can be used to navigate are:
 
-For routing to the `SplashScreen`:
+For routing to the `splashScreen`:
 
 ```
-  static const String splashScreen = '/splashScreen';
+static const String splashScreen = '/splashScreen';
 ```
 
-For routing to the `Introduction`:
+For routing to the `introduction`:
 
 ```
-  static const String introduction = '/introduction';
+static const String introduction = '/introduction';
 ```
 
-For routing to the `HomeEntry`:
+For routing to the `home`:
 
 ```
-  static const String home = '/home';
+static const String home = '/home';
 ```
 
 If you don't want a SplashScreen in your application set your initialRoute to `Introduction`:
@@ -73,42 +78,64 @@ final GoRouter _router = GoRouter(
 );
 ```
 
-To use the module within your Flutter-application without predefined `Go_router` routes but with `Navigator` routes add the following :
+## Navigator
-
-Add the following configuration to your flutter_application:
 
+Add the following code to the build-method of a chosen widget like so:
 ```
-StartUserStoryConfiguration startUserStoryConfiguration = const StartUserStoryConfiguration();
+class Example extends StatelessWidget {
+  const Example({super.key});
+
+  @override
+  Widget build(BuildContext context) {
+    return NavigatorStartUserStory(
+      onComplete: (context) {},
+    );
+  }
+}
 ```
 
-Add the following code to the build-method of a chosen widget:
+or with options:
+
+Place the following code somewhere in your project where it can be accessed globally:
 
 ```
-startNavigatorUserStory(startUserStoryConfiguration, context);
+var startUserStoryConfiguration = const StartUserStoryConfiguration();
 ```
 
-If the splashScreenBuilder is not used the SplashScreen will be skipped.
+```
+class Example extends StatelessWidget {
+  const Example({super.key});
+
+  @override
+  Widget build(BuildContext context) {
+    return NavigatorStartUserStory(
+      configuration: startUserStoryConfiguration,
+      onComplete: (context) {},
+    );
+  }
+}
+```
 
 The `StartUserStoryConfiguration` has its own parameters, as specified below:
 | Parameter | Explanation |
 |-----------|-------------|
-| splashScreenBuilder | The builder for the splashScreen. |
+| `splashScreenBuilder` | The builder to override the default splashScreen |
-| introductionBuilder | The builder for the introduction. |
+| `introductionBuilder` | A builder to wrap the introductions in your own page |
-| introductionOptionsBuilder | The options for the introduction. |
+| `introductionOptionsBuilder` | The builder to override the default `introductionOptions` |
-| introductionService | The service for the introduction. Default IntroductionService (SharedPreferencesIntroductionDataProvider()) |
+|`introductionService` | The service to override the default `introductionService` |
-| homeEntry | The widget that will be shown after the introduction. |
+| `homeScreenRoute` | The route to navigate to after the introduction or splashScreen is completed |
-| homeScreenRoute | The route that will be shown after the introduction when using gorouter |
+| `introductionFallbackScreen` | The screen that is shown when something goes wrong fetching data for the introduction |
-| introductionFallbackScreen | The widget that will be shown when the introduction is skipped. |
+| `introductionScrollPhysics` | The scroll physics for the introduction |
-| introductionScrollPhysics | The scrollPhysics for the introduction. |
+| `showIntroduction` | A boolean to show the introduction or not. Defaults to true |
-| showIntroduction | Whether or not the introduction should be shown. |
+| `useKillswitch` | A boolean to use the killswitch or not. Defaults to false |
-| useKillswitch | Whether or not the killswitch should be used. This will only work when you use the splashScreen and you need to have a active internet connection|
+| `minimumSplashScreenDuration` | The minimum duration the splashScreen should be shown. Defaults to 3 seconds |
-| minimumSplashScreenDuration | The minimum duration the splashScreen should be shown. |
+| `splashScreenFuture` | The future to be completed before the splashScreen is completed |
-| splashScreenFuture | The future that will be awaited before the splashScreen is closed. |
+| `splashScreenCenterWidget` | The widget to be shown in the center of the splashScreen |
-| splashScreenCenterWidget | The widget that will be shown in the center of the splashScreen. |
+| `splashScreenBackgroundColor` | The color of the splashScreen background. Defaults to Color(0xff212121) |
-| splashScreenBackgroundColor | The background color of the splashScreen. |
+| `canPopFromIntroduction` | Allow popping from introduction, defaults to true. Defaults to true |
-| canPopFromIntroduction | Whether or not the introduction can be popped. |
+| `killswitchService` | The service to override the default killswitch service |
-| killswitchService | The service for the killswitch. Instead of the default service |
+| `showSplashScreen` | A boolean to show the splashScreen or not. Defaults to true |
-| showSplashScreen | Whether or not the splashScreen should be shown. |
+
 
 ## Issues
 

example/.gitignore

@@ -31,6 +31,9 @@ migrate_working_dir/
 .pub/
 /build/
 
+# platforms
+/web
+
 # Symbolication related
 app.*.symbols
 

example/README.md

@@ -0,0 +1,16 @@
+# example
+
+A new Flutter project.
+
+## Getting Started
+
+This project is a starting point for a Flutter application.
+
+A few resources to get you started if this is your first Flutter project:
+
+- [Lab: Write your first Flutter app](https://docs.flutter.dev/get-started/codelab)
+- [Cookbook: Useful Flutter samples](https://docs.flutter.dev/cookbook)
+
+For help getting started with Flutter development, view the
+[online documentation](https://docs.flutter.dev/), which offers tutorials,
+samples, guidance on mobile development, and a full API reference.

example/lib/main.dart

@@ -39,38 +39,6 @@ StartUserStoryConfiguration config = StartUserStoryConfiguration(
   splashScreenBuilder: (context, onFinish) => SplashScreen(
     onFinish: onFinish,
   ),
-  introductionOptionsBuilder: (ctx) => IntroductionOptions(
-    pages: [
-      const IntroductionPage(
-        title: Text('First page'),
-        text: Text('Wow a page'),
-        graphic: FlutterLogo(),
-      ),
-      const IntroductionPage(
-        title: Text('Second page'),
-        text: Text('Another page'),
-        graphic: FlutterLogo(),
-      ),
-      const IntroductionPage(
-        title: Text('Third page'),
-        text: Text('The final page of this app'),
-        graphic: FlutterLogo(),
-      ),
-    ],
-    introductionTranslations: const IntroductionTranslations(
-      skipButton: 'Skip it!',
-      nextButton: 'Next',
-      previousButton: 'Previous',
-      finishButton: 'To the app!',
-    ),
-    tapEnabled: true,
-    displayMode: IntroductionDisplayMode.multiPageHorizontal,
-    buttonMode: IntroductionScreenButtonMode.text,
-    indicatorMode: IndicatorMode.dash,
-    skippable: true,
-    buttonBuilder: (context, onPressed, child, type) =>
-        ElevatedButton(onPressed: onPressed, child: child),
-  ),
 );
 
 class SplashScreen extends StatefulWidget {
@@ -114,17 +82,3 @@ class HomeEntry extends StatelessWidget {
         body: const Center(child: Text('HomeEntry')),
       );
 }
-
-class ExampleIntroductionDataProvider
-    implements SharedPreferencesIntroductionDataProvider {
-  @override
-  String key = 'example';
-
-  @override
-  Future<void> setCompleted({bool value = true}) async =>
-      // ignore: void_checks
-      false;
-
-  @override
-  Future<bool> shouldShow() async => true;
-}

example/pubspec.yaml

@@ -12,10 +12,8 @@ dependencies:
   flutter_start:
     path: ../
   flutter_introduction_shared_preferences:
-    git:
+    hosted: https://forgejo.internal.iconica.nl/api/packages/internal/pub/
-      url: https://github.com/Iconica-Development/flutter_introduction
+    version: ^5.0.0
-      ref: 3.0.0
-      path: packages/flutter_introduction_shared_preferences
 
 dev_dependencies:
   flutter_test:

lib/src/user_stories/flutter_start_userstory_go_router.dart

@@ -24,7 +24,6 @@ List<GoRoute> getStartStoryRoutes({
         pageBuilder: (context, state) {
           var go = context.go;
           var isAllowedToPassThrough = false;
-          var introductionSeen = false;
           String? routeAfterSplash;
           Future<void> splashLoadingMethod() async {
             await Future.wait<void>(
@@ -40,11 +39,6 @@ List<GoRoute> getStartStoryRoutes({
                           await killswitchService.isKillswitchActive();
                     }
 
-                    var introService = configuration.introductionService ??
-                        IntroductionService(
-                          SharedPreferencesIntroductionDataProvider(),
-                        );
-                    introductionSeen = !await introService.shouldShow();
                     if (context.mounted)
                       routeAfterSplash = await configuration.splashScreenFuture
                               ?.call(context) ??
@@ -62,7 +56,7 @@ List<GoRoute> getStartStoryRoutes({
 
             if (configuration.useKillswitch && isAllowedToPassThrough) return;
 
-            if (!configuration.showIntroduction || introductionSeen) {
+            if ((!configuration.showIntroduction) && context.mounted) {
               return go(
                 routeAfterSplash ?? StartUserStoryRoutes.home,
               );
@@ -85,7 +79,7 @@ List<GoRoute> getStartStoryRoutes({
                   body: Center(
                     child:
                         configuration.splashScreenCenterWidget?.call(context) ??
-                            defaultSplashScreen,
+                            defaultSplashScreen(context),
                   ),
                 ),
           );

lib/src/user_stories/flutter_start_userstory_navigator.dart

@@ -19,13 +19,11 @@ class NavigatorStartUserStory extends StatelessWidget {
   final void Function(BuildContext context) onComplete;
 
   @override
-  Widget build(BuildContext context) {
+  Widget build(BuildContext context) => Navigator(
-    if (!configuration.showSplashScreen) {
+        onGenerateInitialRoutes: (_, __) => [
-      return _introduction(configuration, context, onComplete);
+          _getInitialRoute(configuration, onComplete),
-    }
+        ],
-
+      );
-    return _splashScreen(configuration, context, onComplete);
-  }
 }
 
 /// Enter the start user story with the Navigator 1.0 API.
@@ -42,6 +40,15 @@ Future<void> startNavigatorUserStory(
   StartUserStoryConfiguration configuration, {
   required void Function(BuildContext context) onComplete,
 }) async {
+  var initialRoute = _getInitialRoute(configuration, onComplete);
+
+  await Navigator.of(context).push(initialRoute);
+}
+
+MaterialPageRoute<dynamic> _getInitialRoute(
+  StartUserStoryConfiguration configuration,
+  void Function(BuildContext context) onComplete,
+) {
   var initialRoute = MaterialPageRoute(
     builder: (context) => _splashScreen(
       configuration,
@@ -59,8 +66,7 @@ Future<void> startNavigatorUserStory(
       ),
     );
   }
-
+  return initialRoute;
-  await Navigator.of(context).push(initialRoute);
 }
 
 Widget _splashScreen(
@@ -71,7 +77,6 @@ Widget _splashScreen(
   var navigator = Navigator.of(context);
 
   var isAllowedToPassThrough = false;
-  var introductionSeen = false;
 
   Future<void> splashHandler() async {
     await Future.wait<void>(
@@ -87,12 +92,6 @@ Widget _splashScreen(
               isAllowedToPassThrough =
                   await killswitchService.isKillswitchActive();
             }
-
-            var introService = configuration.introductionService ??
-                IntroductionService(
-                  SharedPreferencesIntroductionDataProvider(),
-                );
-            introductionSeen = !await introService.shouldShow();
           },
         ),
         Future.delayed(
@@ -105,34 +104,37 @@ Widget _splashScreen(
 
     if (configuration.useKillswitch && isAllowedToPassThrough) return;
 
-    if ((!configuration.showIntroduction || introductionSeen) &&
+    var introService = configuration.introductionService ??
-        context.mounted) {
+        IntroductionService(SharedPreferencesIntroductionDataProvider());
-      onComplete(context);
-      return;
-    }
 
-    if (context.mounted) {
+    var shouldShowIntroduction =
-      await navigator.pushReplacement(
+        configuration.showIntroduction && await introService.shouldShow();
-        MaterialPageRoute(
+
-          builder: (context) => _introduction(
+    if (!context.mounted) return;
-            configuration,
+
-            context,
+    if (!shouldShowIntroduction) return onComplete(context);
-            onComplete,
+
-          ),
+    await navigator.pushReplacement(
+      MaterialPageRoute(
+        builder: (context) => _introduction(
+          configuration,
+          context,
+          onComplete,
         ),
-      );
+      ),
-    }
+    );
   }
 
+  unawaited(splashHandler());
+
   var builder = configuration.splashScreenBuilder;
 
   if (builder == null) {
-    unawaited(splashHandler());
     return Scaffold(
       backgroundColor: configuration.splashScreenBackgroundColor,
       body: Center(
         child: configuration.splashScreenCenterWidget?.call(context) ??
-            defaultSplashScreen,
+            defaultSplashScreen(context),
       ),
     );
   }

lib/src/widgets/default_splash_screen.dart

@@ -1,10 +1,6 @@
 import 'package:flutter/material.dart';
 
-const defaultSplashScreen = Text(
+Text defaultSplashScreen(BuildContext context) => Text(
-  'Splash Screen',
+      'iconinstagram',
-  style: TextStyle(
+      style: Theme.of(context).textTheme.headlineLarge,
-    color: Color(0xff71C6D1),
+    );
-    fontWeight: FontWeight.w800,
-    fontSize: 32,
-  ),
-);

pubspec.yaml

@@ -1,7 +1,8 @@
 name: flutter_start
 description: "Flutter_start is a package that allows you to jumpstart your application with a splashScreen, introduction and a home."
-publish_to: "none"
+version: 4.2.4
-version: 4.0.0
+
+publish_to: https://forgejo.internal.iconica.nl/api/packages/internal/pub
 
 environment:
   sdk: ">=3.2.5 <4.0.0"
@@ -9,20 +10,16 @@ environment:
 dependencies:
   flutter:
     sdk: flutter
-  cupertino_icons: ^1.0.2
+  cupertino_icons: ">=1.0.2 <2.0.0"
-  go_router: any
+  go_router: ">=14.2.0 <15.0.0"
-  http: any
+  http: ">=1.2.1 <2.0.0"
-  package_info_plus: any
+  package_info_plus: ">=8.0.0 <9.0.0"
   flutter_introduction:
-    git:
+    hosted: https://forgejo.internal.iconica.nl/api/packages/internal/pub
-      url: https://github.com/Iconica-Development/flutter_introduction
+    version: ^5.0.0
-      ref: 3.0.0
-      path: packages/flutter_introduction
   flutter_introduction_shared_preferences:
-    git:
+    hosted: https://forgejo.internal.iconica.nl/api/packages/internal/pub/
-      url: https://github.com/Iconica-Development/flutter_introduction
+    version: ^5.0.0
-      ref: 3.0.0
-      path: packages/flutter_introduction_shared_preferences
 
 dev_dependencies:
   flutter_test:

test/_test.dart

@@ -0,0 +1,14 @@
+// This is an example unit test.
+//
+// A unit test tests a single function, method, or class. To learn more about
+// writing unit tests, visit
+// https://flutter.dev/docs/cookbook/testing/unit/introduction
+import 'package:flutter_test/flutter_test.dart';
+
+void main() {
+  group('Plus Operator', () {
+    test('should add two numbers together', () {
+      expect(1 + 1, 2);
+    });
+  });
+}