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

Number of errors, warning, etc.. is always 0 for merged data sources #225

Closed
Kittyfisto opened this issue Nov 6, 2019 · 1 comment · Fixed by #248
Closed

Number of errors, warning, etc.. is always 0 for merged data sources #225

Kittyfisto opened this issue Nov 6, 2019 · 1 comment · Fixed by #248
Labels

Comments

@Kittyfisto
Copy link
Owner

Current behaviour

Merging multiple log files causes the display at the top to display 0 errors, 0 warnings, etc...
image

Expected behaviour

Same as with simple data sources.

Steps to reproduce the problem

Merge a few files together.

@Kittyfisto Kittyfisto added the bug label Nov 6, 2019
Kittyfisto added a commit that referenced this issue Dec 10, 2020
…225

The data sources view model would never call Update() on newly created groups. This caused values such as number of errors, warnings, etc.. to never be updated and remain at 0 instead.
The bug would only occur when the grouped data source was created during the session however, e.g. closing tailviewer and opening it again would then show the correct number of warnings 100% of the time.
Kittyfisto added a commit that referenced this issue Dec 10, 2020
…225 (#248)

The data sources view model would never call Update() on newly created groups. This caused values such as number of errors, warnings, etc.. to never be updated and remain at 0 instead.
The bug would only occur when the grouped data source was created during the session however, e.g. closing tailviewer and opening it again would then show the correct number of warnings 100% of the time.
@Kittyfisto
Copy link
Owner Author

Fixed with Tailviewer v0.9.3.893.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant