Skip to content

Commit 1af2e21

Browse files
authored
Merge pull request #161 from federicoviceconti/main
Fix CircularProgressIndicator never shown on first load
2 parents 9f90a2f + 37fc0e3 commit 1af2e21

File tree

2 files changed

+9
-2
lines changed

2 files changed

+9
-2
lines changed

lib/src/modules/fvm/fvm.screen.dart

+3-2
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,10 @@ class FVMScreen extends HookWidget {
1616

1717
@override
1818
Widget build(BuildContext context) {
19-
final cachedVersions = useProvider(releasesStateProvider).all;
19+
final appReleaseState = useProvider(releasesStateProvider);
20+
final cachedVersions = appReleaseState.all;
2021

21-
if (cachedVersions == null) {
22+
if (appReleaseState.fetching) {
2223
return const Center(child: CircularProgressIndicator());
2324
}
2425

lib/src/modules/releases/releases.provider.dart

+6
Original file line numberDiff line numberDiff line change
@@ -10,15 +10,19 @@ import '../common/constants.dart';
1010
import '../fvm/fvm.provider.dart';
1111

1212
class AppReleasesState {
13+
bool fetching;
14+
1315
MasterDto master;
1416
List<ChannelDto> channels;
1517
List<VersionDto> versions;
1618

1719
bool hasGlobal;
20+
1821
AppReleasesState({
1922
this.channels,
2023
this.versions,
2124
this.master,
25+
this.fetching = true,
2226
this.hasGlobal = false,
2327
}) {
2428
channels = <ChannelDto>[];
@@ -67,6 +71,8 @@ final releasesStateProvider = Provider<AppReleasesState>((ref) {
6771
return releasesState;
6872
}
6973

74+
releasesState.fetching = false;
75+
7076
final flutterReleases = payload.releases;
7177
final flutterChannels = payload.channels;
7278

0 commit comments

Comments
 (0)