Skip to content

fix: 7373 - Initialize counters on app bar statistics card load#7403

Open
maxweilun1989 wants to merge 1 commit into
openfoodfacts:developfrom
maxweilun1989:fix/appbar_loading
Open

fix: 7373 - Initialize counters on app bar statistics card load#7403
maxweilun1989 wants to merge 1 commit into
openfoodfacts:developfrom
maxweilun1989:fix/appbar_loading

Conversation

@maxweilun1989
Copy link
Copy Markdown

Ensure statistics counters are loaded when the AppBarStatisticsCard is first created, addressing the issue where counters may not be initialized properly after login.

⚠️ Acceptance Requirements

This Pull Request will only be accepted if:

  • ✅ It is linked to an issue (using linking keywords like Fixes, Closes, or Resolves)
  • ✅ The linked issue is assigned to the author of this PR

What

  • Added an initState method to _AppBarStatisticsCardState that calls _asyncLoad() when the widget is first initialized

Screenshot or video

screen-20260205-214535.mp4

Fixes bug(s)

Part of

Ensure statistics counters are loaded when the AppBarStatisticsCard
is first created, addressing the issue where counters may not be
initialized properly after login.
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Feb 5, 2026

Codecov Report

❌ Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 9.12%. Comparing base (4d9c7fc) to head (a1a29eb).
⚠️ Report is 1367 commits behind head on develop.

Files with missing lines Patch % Lines
...d_in/statistics_cards/app_bar_statistics_card.dart 0.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           develop   #7403      +/-   ##
==========================================
- Coverage     9.54%   9.12%   -0.43%     
==========================================
  Files          325     624     +299     
  Lines        16411   36586   +20175     
==========================================
+ Hits          1567    3337    +1770     
- Misses       14844   33249   +18405     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@teolemon teolemon added ✏️ Contribution 👥 User management Account login, signup, signout labels Feb 27, 2026
description: This file stores settings for Dart & Flutter DevTools.
documentation: https://docs.flutter.dev/tools/devtools/extensions#configure-extension-enablement-states
extensions:
- provider: true No newline at end of file
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is this change about ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✏️ Contribution 👥 User management Account login, signup, signout

Projects

Status: 💬 To discuss and validate

Development

Successfully merging this pull request may close these issues.

Counters not initialized upon connection

3 participants