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

Fix runtime crash #620

Merged
merged 8 commits into from
Aug 31, 2024
Merged

Fix runtime crash #620

merged 8 commits into from
Aug 31, 2024

Conversation

meiron03
Copy link
Member

  • Fix dependency issues
  • Add try/catch statements to prevent app from crashing when network requests fail
  • Additional bug fixes related to widgets, laundry, and dining
  • Update ktlint to 1.3.1

meiron03 and others added 8 commits August 22, 2024 18:05
Two imports from jCenter (com.eightbitlab:supportrenderscriptblur:1.0.2
and com.eightbitlab:blurview:1.5.0) no longer exist which was causing
the app to no longer build. I removed the first import since we no longer
use this library. The second module was used by
'com.github.searchy2:CustomAlertViewDialogue:2.6.1' so I excluded it and
instead added the newer version which is on jitpack. Also, jCenter was
removed from repositories since it has been deprecated for a while and we
no longer need it. Finally, dependencies in build.gradle were sorted for
readability.
… declared as a separate function, the tab is not set to dining when widget is clicked.
Somehow one of the libraries broke, so I reverted to an older version.
Copy link
Contributor

@baronhsieh2005 baronhsieh2005 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The program is running smoothly without fatal crashes when runtime exceptions are thrown and the NoSuchElementExceeption is also no longer being thrown after the LaundryViewModel logic change (not removing an empty rooms list anymore). Laundry functionality might need to be further tested after the backend issues are resolved but currently our app is back to running normally. Thank you so much Aaron!

@meiron03 meiron03 merged commit 1ca079a into main Aug 31, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants