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

Notes for incomplete days are not shown if frequency target is already met #1566

Closed
wolfgangkarall opened this issue Oct 31, 2022 · 1 comment
Labels
bug Something is not behaving as the developers expected confirmed The developers have been able to confirm that the issue is valid

Comments

@wolfgangkarall
Copy link

Pre-submission checklist

  • [ x ] I am submitting a bug report, not a feature request.
  • [ x ] I have searched for similar issues, but did not find any matches.

Description

If a habit already has its frequency target met (e.g. '4 times per week'), it is not possible to see a note for an incomplete day (marked 'x' in the pop-up). It does work when marked complete (tick in the pop-up) or skipped ('-' in the pop-up).

Looking at the sqlite3 DB that's the backup, I see the text entered there in the 'Repetitions' table. It's not in the CSV export.

It's not visible in the habits overview screen

image

I'd expect an 'x' with the dot instead of the unfilled tick, as I see 'x' as manual entry, while the unfilled tick as sign of 'this day has no entry, but the frequency target is met already', but an unfilled tick with a dot would probably work just as well.

But what's worse, the text is gone from the pop-up when editing that day's status again.

There's also no visual hint in the calendar view:

image

Steps to reproduce

  1. create some habit with a frequency like '4 times a week'
  2. mark that habit with a tick for 4 days
  3. add a note on day 5 while also marking it incomplete ('x')
  4. open the entry's pop-up again -> there is no note

System information

  • Phone: Mi A1
  • Phone Operating System: Android 9
  • App version: latest F-Droid version, 2.1.1

Screenshots

see above

@hiqua hiqua added bug Something is not behaving as the developers expected confirmed The developers have been able to confirm that the issue is valid labels Oct 31, 2022
iSoron added a commit that referenced this issue Jan 24, 2023
@iSoron
Copy link
Owner

iSoron commented Jan 24, 2023

Fixed in 2131fb3. The fix should become available in Loop 2.1.2.

@iSoron iSoron closed this as completed Jan 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is not behaving as the developers expected confirmed The developers have been able to confirm that the issue is valid
Projects
None yet
Development

No branches or pull requests

3 participants