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

Incorrect Timezone in Code #303

Closed
7 tasks done
bartoshevich opened this issue Aug 22, 2024 · 2 comments · Fixed by #307
Closed
7 tasks done

Incorrect Timezone in Code #303

bartoshevich opened this issue Aug 22, 2024 · 2 comments · Fixed by #307
Labels
bug Something is not working

Comments

@bartoshevich
Copy link

Checklist

  • I can reproduce the bug with the latest version given here.
  • I made sure that there are no existing issues - open or closed - to which I could contribute my information.
  • I made sure that there are no existing discussions - open or closed - to which I could contribute my information.
  • I have read the FAQs inside the app (Menu -> About -> FAQs) and my problem isn't listed.
  • I have taken the time to fill in all the required details. I understand that the bug report will be dismissed otherwise.
  • This issue contains only one bug.
  • I have read and understood the contribution guidelines.

Affected app version

1.0.3

Affected Android/Custom ROM version

Android 14

Affected device model

Pixel 8

How did you install the app?

F-Droid / IzzyOnDroid

Steps to reproduce the bug

  1. Navigate to app/src/main/kotlin/org/fossify/calendar/helpers/MyTimeZones.kt

  2. Note that the expected GMT offset for "Asia/Almaty" should be GMT+5 (since 2024), not GMT+6

Expected behavior

The correct association should be "GMT+5" for "Asia/Almaty"

Actual behavior

The code incorrectly associates "Asia/Almaty" with "GMT+6", which results in an incorrect time calculation.

Screenshots/Screen recordings

изображение

Additional information

No response

@bartoshevich bartoshevich added bug Something is not working needs triage Issue is not yet ready for PR authors to take up labels Aug 22, 2024
@Aga-C Aga-C removed the needs triage Issue is not yet ready for PR authors to take up label Aug 22, 2024
nofishonfriday added a commit to nofishonfriday/Fossify-Calendar that referenced this issue Aug 25, 2024
nofishonfriday added a commit to nofishonfriday/Fossify-Calendar that referenced this issue Aug 25, 2024
@igoose1
Copy link

igoose1 commented Aug 29, 2024

Hi, I second it! I spent two hours trying to figure out why my computer's events delay for an hour compared to my phone's event (I use CalDAV), and I'm finally happy to find a reason! I'm indeed in Almaty currently, my computer's tzdata is correct and it seems an issue is in a manually updated app/src/main/kotlin/org/fossify/calendar/helpers/MyTimeZones.kt file.

This PR seems okay for now: https://github.com/FossifyOrg/Calendar/pull/307/files

A better PR would update all timezones: https://www.joda.org/joda-time/timezones.html

The best PR would replace self-hosted timezone database with java.util.TimeZone: https://developer.android.com/reference/java/util/TimeZone

I've never programmed for Android, although I can help to investigate how to do this right.

nofishonfriday added a commit to nofishonfriday/Fossify-Calendar that referenced this issue Sep 11, 2024
naveensingh added a commit that referenced this issue Nov 3, 2024
@naveensingh
Copy link
Member

Should be fixed in the next update.

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

Successfully merging a pull request may close this issue.

4 participants