Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix/ LateInitializationError: Field 'prefs' has not been initialized. #1374

Merged
merged 2 commits into from
Oct 18, 2024

Conversation

ghassenbenzahra123
Copy link
Contributor

📝 Summary

This PR fixes #1372

Description

Make sure to initialize shared prefs before calling fetchmosque in mosqueManager.

Checklist:

  • Coding Standards: I have reviewed my code to ensure it follows the project's coding standards.
  • Testing: I have tested the changes and they work as expected.
  • Merge Conflicts: I have resolved any merge conflicts with the latest main/development branch.
  • Branch Status: The branch is up-to-date with the target branch (main/development).

@ghassenbenzahra123 ghassenbenzahra123 marked this pull request as ready for review October 16, 2024 17:01
@YassinNouh21
Copy link
Contributor

YassinNouh21 commented Oct 17, 2024

@ghassenbenzahra123
it is called from the Api.dart not the mosque_manager.dart. it is correct that it happens at the msoque_manager.dart but even though the cause error is at Api.dart

@YassinNouh21 YassinNouh21 merged commit 5914943 into main Oct 18, 2024
1 check passed
@YassinNouh21 YassinNouh21 deleted the fix/late-pref-init-mosque-manager branch October 18, 2024 12:44
@YassinNouh21 YassinNouh21 added the bug Something isn't working label Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

LateError: LateInitializationError: Field 'prefs' has not been initialized.
2 participants