TUM-Dev / campus_flutter

Development of the cross-platform TUM Campus App with Flutter - for and from students at Technical University of Munich.
https://tum.app/
GNU General Public License v3.0
34 stars 4 forks source link

[BUG] Decoding of Calendar Events Fails on Android #244

Closed benjibst closed 4 months ago

benjibst commented 4 months ago

Describe the bug The calendar widget doesn't show any lectures, even though i have 4 lectures today that are displayed in the calendar within the app.

To Reproduce Steps to reproduce the behavior: Add the calendar widget to the homescreen of an android device

Expected behavior A list of lectures of the current and following days should be displayed by the widget.

Screenshots Screenshot_2024-04-15-10-00-34-59_b783bf344239542886fee7b48fa4b892.jpg

Smartphone (please complete the following information):

jakobkoerber commented 4 months ago

I'm still investigating this... this is really hard to reproduce on Android since everybody uses different devices and Android distribution. Which campus app version are you currently using? And do still experience crashes or just an empty widget?

Marlene163 commented 4 months ago

I have the same problem (on OnePlus Nord CE 5G: Android 12 and Samsung Tab 8: Android 14)

App Version: 5.0.1 und 5.0.2

benjibst commented 4 months ago

There are still crashes (as far as i can tell only with the calendar widget active) and im using the version 5.0.2 according to my phones app settings. its the most up to date version on Play Store with Beta tester enabled.

jakobkoerber commented 4 months ago

@benjibst please download the latest internal beta version (5.0.3 Build 1017) and test if the issues still persist... I invested a lot of time attempting to debug the issue 😅 hopefully it is now resolved

CommanderStorm commented 4 months ago

please download the latest internal beta version

No such beta exists on my device. Is this expected? (I am asking because https://github.com/TUM-Dev/campus_flutter/pull/246/commits/23367be54e756d57d3ebf229f6e88275386a6372 had some build errors)

benjibst commented 4 months ago

@benjibst please download the latest internal beta version (5.0.3 Build 1017) and test if the issues still persist... I invested a lot of time attempting to debug the issue 😅 hopefully it is now resolved

Hello Jakob, Unfortunately the issue persists on my device:( the calendar widget still doesnt show any events and when i open the app with the widget placed, it always crashes.

jakobkoerber commented 4 months ago

please download the latest internal beta version

No such beta exists on my device. Is this expected?

(I am asking because https://github.com/TUM-Dev/campus_flutter/pull/246/commits/23367be54e756d57d3ebf229f6e88275386a6372 had some build errors)

I think you're not part of the internal beta track (yet) 😄

jakobkoerber commented 4 months ago

@benjibst please download the latest internal beta version (5.0.3 Build 1017) and test if the issues still persist... I invested a lot of time attempting to debug the issue 😅 hopefully it is now resolved

Hello Jakob, Unfortunately the issue persists on my device:( the calendar widget still doesnt show any events and when i open the app with the widget placed, it always crashes.

That's really frustrating, since it takes a lot of time to even reproduce such issues 🙃 could you confirm the version and build number for me, by navigating to the settings within the app? Did you try to re-install the app at some point? And would you be willing to provide me with your TUM ID, e.g. via app@tum.de? I would like to request a token for further debugging, activating the calendar permission should be sufficient :)

benjibst commented 4 months ago

Yes, i confirm i have build 1017 installed on my device Screenshot_2024-04-21-09-18-16-19_6e0b122dca11de86c8dd96dbad655ddf.jpg

Also, the issue persists after reinstalling the application, if anything it got worse because now it crashes even without the widget.

My tum id is go73nej

jakobkoerber commented 4 months ago

@benjibst I rewrote a lot of things in the Android widget and published them as an internal beta with 5.0.3 (1018), please delete the app, download the latest version and see if the issue still persists 😅 thanks a lot for your help!

benjibst commented 4 months ago

@benjibst I rewrote a lot of things in the Android widget and published them as an internal beta with 5.0.3 (1018), please delete the app, download the latest version and see if the issue still persists 😅 thanks a lot for your help!

Hello Jakob, downloaded the new build, it seems to work now! Thank you!

jakobkoerber commented 4 months ago

That's great to hear! I hope it stays that way 😄 thanks for your cooperation and patience!

benjibst commented 4 months ago

Another thing that would be nice, to make the widget more useful: To show when an event is due, use the day name and day number within month, instead of day number and month name. Like in the google calendar widget for example. Since i believe the widget is used mainly to look up appointments within a given day or the next few days, i think this would be useful. Otherwise you have to do the maths in your head. Screenshot_2024-04-21-15-55-34-15_b783bf344239542886fee7b48fa4b892.jpg

Added a screenshot of how this looks in the google calendar widget