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

Hidden rows do not show time entires, do not contribute to sums #491

Closed
kusalananda opened this issue May 24, 2022 · 0 comments · Fixed by #493
Closed

Hidden rows do not show time entires, do not contribute to sums #491

kusalananda opened this issue May 24, 2022 · 0 comments · Fixed by #493
Labels
bug Something isn't working frontend This is likely affecting the frontend

Comments

@kusalananda
Copy link
Member

kusalananda commented May 24, 2022

Describe the bug

  • Hidden rows do not show time entries.
  • The time entries on hidden rows are not used when calculating sums of hours.
  • When you unhide a newly logged hidden time entry, the time records disappear

Steps to reproduce

  1. Enter time on a row that is hidden.
  2. Save changes.
  3. Reload.
  4. Observe the sum total not changing and no time entries showing on the hidden rows.

Entries are saved and will be visible etc. when the row is un-hidden and the page is reloaded.

Environment

Current develop branch, just after merging #490.

Expected behaviour

When the section with hidden rows is expanded, it should function identically to a starred or other rows. Time entries may be added or modified etc.

The hidden rows should always be used when summing up the hours.

@kusalananda kusalananda added bug Something isn't working frontend This is likely affecting the frontend labels May 24, 2022
@ghost ghost closed this as completed in #493 May 25, 2022
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working frontend This is likely affecting the frontend
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant