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

Feature/edured 78 delete login details #73

Merged
merged 6 commits into from
Apr 14, 2023

Conversation

IuliaSTANA
Copy link
Contributor

Edured-78: Add implementation for remove service
Remove unrequired Companion object wrapper for Provider.
Make sure the data and activity screen can show loading state & error messages. Refactored screen to use the existing top app bar.
Navigate to remove service warning screen from Data & Activity screen. Grouped navigation graph destinations.
Update dependencies.
Treat delete service as part of the data and activity screen content and not as an independent screen.
Ensure compatibility between compose & kotlin compiler

Iulia Stana added 6 commits April 12, 2023 09:24
Remove unrequired Companion object wrapper for Provider.
Make sure the data and activity screen can show loading state & error messages. Refactored screen to use the existing top app bar.
Navigate to remove service warning screen from Data & Activity screen. Grouped navigation graph destinations.
…at tokens are being queried and passed for the service to be deleted
…and not as an independent screen.

Ensure compatibility between compose & kotlin compiler
https://developer.android.com/jetpack/androidx/releases/compose-compiler#version_145_3

Trying to fix
"e: java.lang.UnsupportedClassVersionError: ***x/compose/compiler/plugins/kotlin/ComposeComponentRegistrar has been compiled by a more recent version of the Java Runtime (class file version 61.0), this version of the Java Runtime only recognizes class file versions up to 55.0"
happening on github actions.
@IuliaSTANA IuliaSTANA merged commit 2a7b538 into develop Apr 14, 2023
@IuliaSTANA IuliaSTANA deleted the feature/edured-78_delete_login_details branch April 14, 2023 14:24
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.

1 participant