Closed danielsitnik closed 4 days ago
⚠️ The automated review process is experimental and likely has bugs. Please bear with us as we iron out the kinks and enable you to ship changes at high velocity 🚀
Hello! Thank you so much for your change 🤜 🤛 . There are a few things you need to do:
Hang tight! A Tidbyt engineer will be by shortly to review your change. Here is what they will be looking for: | Test | Details | |
---|---|---|---|
🟡 | App Dir | Touches files in multiple directories | |
🟡 | Modules | Multiple .star files: apps/company_clock/company_clock.star, apps/gradientclock2/gradientclock2.star | |
🟡 | Original Author | The original author (qqvq-d) does not match the PR author (danielsitnik) |
LGTM. @qqvq-d ?
Yes, please go ahead
On Wed, Jul 3, 2024 at 5:11 PM Mats Linander @.***> wrote:
LGTM. @qqvq-d https://github.com/qqvq-d ?
— Reply to this email directly, view it on GitHub https://github.com/tidbyt/community/pull/2527#issuecomment-2207304822, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALFGJ3VCHIZIOYL7LWLSJMTZKRSHBAVCNFSM6AAAAABKEQFNK6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBXGMYDIOBSGI . You are receiving this because you were mentioned.Message ID: @.***>
This fixes an issue with incorrect timezone handling on the app, as reported by the user candido in the forums: https://discuss.tidbyt.com/t/the-time-on-the-new-clock-apps-are-two-hours-wrong/6543
@qqvq-d we need your approval here.