Fetched all snippets at once

Tomasz Kądziołka · Tomasz Kądziołka · commit de33d9d49a1b · 2022-11-27T22:30:02.000+01:00
diff --git a/app/src/main/java/pl/tkadziolka/snipmeandroid/bridge/detail/DetailModel.kt b/app/src/main/java/pl/tkadziolka/snipmeandroid/bridge/detail/DetailModel.kt
@@ -86,9 +86,9 @@ class DetailModel(
                         setState(Loaded(it))
                         mutableEvent.value = Saved(saved.uuid)
                     },
-                    onError = {
-                        Timber.e("Couldn't load snippets, error = $it")
-                        parseError(it)
+                    onError = { error ->
+                        Timber.e("Couldn't save snippet, error = $error")
+                        parseError(error)
                     }
                 ).also { disposables += it }
         }
diff --git a/app/src/main/java/pl/tkadziolka/snipmeandroid/bridge/main/MainModel.kt b/app/src/main/java/pl/tkadziolka/snipmeandroid/bridge/main/MainModel.kt
@@ -119,6 +119,7 @@ class MainModel(
             .subscribeBy(
                 onSuccess = {
                     mutableState.value = (Loaded(user, it, pages, scope))
+                    loadNextPage()
                     if (shouldRefresh) {
                         mutableEvent.value = ListRefreshed
                         shouldRefresh = false
diff --git a/app/src/main/java/pl/tkadziolka/snipmeandroid/domain/snippet/CreateSnippetUseCase.kt b/app/src/main/java/pl/tkadziolka/snipmeandroid/domain/snippet/CreateSnippetUseCase.kt
@@ -13,6 +13,7 @@ class CreateSnippetUseCase(
     private val snippetRepository: SnippetRepository
 ) {
     // TODO Test saving other snippet in app and navigation
+    // TODO ERROR: We can still click save saved snip
     operator fun invoke(
         title: String,
         code: String,
diff --git a/flutter_module/lib/presentation/screens/details_screen.dart b/flutter_module/lib/presentation/screens/details_screen.dart
@@ -77,6 +77,7 @@ class _DetailsPage extends HookWidget {
         return const SizedBox();
       }
 
+      navigator.back();
       model.resetEvent();
       navigator.goToDetails(context, snippetId);
     }
diff --git a/flutter_module/lib/presentation/screens/main_screen.dart b/flutter_module/lib/presentation/screens/main_screen.dart
@@ -93,7 +93,7 @@ class _MainPage extends HookWidget {
         },
       ),
       floatingActionButton: FloatingActionButton(
-        onPressed: () => detailsNavigator.goToDetails(context, Mocks.snippet.uuid!),
+        onPressed: () => model.loadNextPage(),
         tooltip: 'Increment',
         child: const Icon(Icons.add),
       ),
diff --git a/flutter_module/lib/presentation/widgets/snippet_action_bar.dart b/flutter_module/lib/presentation/widgets/snippet_action_bar.dart
@@ -44,7 +44,7 @@ class SnippetActionBar extends StatelessWidget {
           StateIcon(
             icon: Icons.save_alt_outlined,
             active: snippet.isSaved,
-            onTap: snippet.isSaved == false ? null : onSaveTap,
+            onTap: snippet.isSaved == false ? onSaveTap : null,
           ),
           const SizedBox(width: Dimens.l),
           StateIcon(

Original file line number	Diff line number	Diff line change
`@@ -86,9 +86,9 @@ class DetailModel(`
`86`	`86`	`setState(Loaded(it))`
`87`	`87`	`mutableEvent.value = Saved(saved.uuid)`
`88`	`88`	`},`
`89`		`- onError = {`
`90`		`- Timber.e("Couldn't load snippets, error = $it")`
`91`		`- parseError(it)`
	`89`	`+ onError = { error ->`
	`90`	`+ Timber.e("Couldn't save snippet, error = $error")`
	`91`	`+ parseError(error)`
`92`	`92`	`}`
`93`	`93`	`).also { disposables += it }`
`94`	`94`	`}`
Original file line number	Diff line number	Diff line change
`@@ -77,6 +77,7 @@ class _DetailsPage extends HookWidget {`
`77`	`77`	`return const SizedBox();`
`78`	`78`	`}`
`79`	`79`
	`80`	`+ navigator.back();`
`80`	`81`	`model.resetEvent();`
`81`	`82`	`navigator.goToDetails(context, snippetId);`
`82`	`83`	`}`