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

Treatements apparently only appear in Today tab if lastTaken is null... #48

Closed
thomas-maeder opened this issue Mar 9, 2024 · 16 comments

Comments

@thomas-maeder
Copy link

...but not if Frequency is Every day and lastTaken refers to yesterday.

@diegopvlk
Copy link
Owner

diegopvlk commented Mar 9, 2024

Try changing the lastUpdate date to yesterday, make sure to restart the app. If you mess up with this file, it'll cause issues.

@thomas-maeder
Copy link
Author

I have restarted many times - doesn't help.

BTW: the application doesn't really terminate if I hit the X button; I have to do Ctrl-C in the shell to really terminate it.

@diegopvlk
Copy link
Owner

diegopvlk commented Mar 9, 2024

It's intentional, so it can send background notifications.

...but not if Frequency is Every day and lastTaken refers to yesterday.

If that it's the only issue that you're having, it should be an easy fix.

@diegopvlk
Copy link
Owner

I pushed an update to snap, see if that fixes the your issues.

@thomas-maeder
Copy link
Author

Thanks. First impression is good!

  • "Donate" isn't translated (to "Spenden") - have I missed it when I did the translations?

  • "Danke für dieses Projekte!" should be "Danke an diese Projekte!" - I must have lacked some context when I did this

  • "Verlassen" - now that I see it, I'd prefer "Beenden" (terminate); on the other hand, the option doesn't really terminate the program, so maybe "Verlassen" (quit) is better...

  • dialog "Behandlung bearbeiten":

    • "Pill(s)" should be "Pille(n)"
    • "Notes" should be "Notiz" (singular) or "Notizen" (plural)
    • "Dosage" should be "Dosierung" - but only here - you probably want to keep "Dosage" as application name - I didn't have enough context when I translated
    • "Erinnerung um" should be "Erinnerung bei" - "um" would be correct if "Notification at" referred to a time, not an amount
  • dialog "Historie":

    • "Pill(s)" should be "Pille(n)"
    • weekdays are not localized
    • date format is not localized
  • dialog "Heute":

    • "Pill(s)" should be "Pille(n)"

The Today/Heute tab lists the expected items right now; let's wait for tomorrow to see if the contents of this tab is correct in the long run.

@diegopvlk
Copy link
Owner

diegopvlk commented Mar 10, 2024

"Donate" isn't translated (to "Spenden") - have I missed it when I did the translations?

You didn't, that one is not translatable.

Feel free to make corrections in the translation, it'll be pushed automatically.

"Dosage" should be "Dosierung" - but only here - you probably want to keep "Dosage" as application name - I didn't have enough context when I translated

It's fine for the app name to be translated.

weekdays are not localized
date format is not localized

These one comes from the system, not the app. If that's not working it's a problem with the locale in your system.

@thomas-maeder
Copy link
Author

These one comes from the system, not the app. If that's not working it's a problem with the locale in your system.

$ env | egrep -i '^(LANG|LC)'
LANGUAGE=de_CH:de
LANG=de_CH.UTF-8

What else is needed?

@diegopvlk
Copy link
Owner

What else is needed?

Try env LC_TIME=de_CH dosage-tracker

@thomas-maeder
Copy link
Author

Try env LC_TIME=de_CH dosage-tracker

Doesn't help - week days are still English, and the date format is still the completely unreadable M/D/Y.

@thomas-maeder
Copy link
Author

Feel free to make corrections in the translation, it'll be pushed automatically.

Done.

@diegopvlk
Copy link
Owner

Doesn't help - week days are still English, and the date format is still the completely unreadable M/D/Y.

what about env LC_ALL=de_CH dosage-tracker?

@thomas-maeder
Copy link
Author

what about env LC_ALL=de_CH dosage-tracker?

Doesn't help, either.

@diegopvlk
Copy link
Owner

You're missing some translations in your system then, it's working for me:
image

@thomas-maeder
Copy link
Author

Hmm - I agree, this looks perfect!

@thomas-maeder
Copy link
Author

Hmm.

$ date +%A
Sonntag

$ date +%c
So 10 Mär 2024 13:22:44

$ date +%x
10.03.2024

@diegopvlk
Copy link
Owner

diegopvlk commented Mar 10, 2024

I have no idea where but you need to set LC_TIME somewhere, closing because this is not related to the app anymore.

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

No branches or pull requests

2 participants