Skip to content

fix: add default values on stat processing#7891

Merged
Alkarex merged 1 commit intoFreshRSS:edgefrom
aledeg:fix/empty-stats
Sep 1, 2025
Merged

fix: add default values on stat processing#7891
Alkarex merged 1 commit intoFreshRSS:edgefrom
aledeg:fix/empty-stats

Conversation

@aledeg
Copy link
Member

@aledeg aledeg commented Sep 1, 2025

Before, there was an error when retrieving stats for a user without feeds. Now, there are default values to display empty stats instead of an exception.

See #7884

Closes #7884

Changes proposed in this pull request:

  • add default values when retrieving stat data

How to test the feature manually:

  1. create a new user
  2. connect as the new user
  3. display stats
  4. validate that there is no errors

Pull request checklist:

  • clear commit messages
  • code manually tested
  • unit tests written (optional if too hard)
  • documentation updated

Additional information can be found in the documentation.

Before, there was an error when retrieving stats for a user without feeds. Now, there
are default values to display empty stats instead of an exception.

See FreshRSS#7884
@Alkarex Alkarex added this to the 1.28.0 milestone Sep 1, 2025
@Alkarex Alkarex added the UI 🎨 User Interfaces label Sep 1, 2025
@Alkarex Alkarex merged commit d31f485 into FreshRSS:edge Sep 1, 2025
1 check passed
@aledeg aledeg deleted the fix/empty-stats branch September 1, 2025 22:18
@Alkarex Alkarex modified the milestones: 1.28.0, 1.27.1 Sep 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

UI 🎨 User Interfaces

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Main statistics page error message

2 participants