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

É só uma lista... #58

Open
wants to merge 156 commits into
base: master
Choose a base branch
from
Open

É só uma lista... #58

wants to merge 156 commits into from

Conversation

wagarcdev
Copy link

No description provided.

migrate dependencies to toml file [version catalog]
migrate gradle files to kotlin script
network security config is only available from api 24+
redundant as was only used in retrofit, which has it embedded
- Capture error messages directly from ResponseBody in Retrofit calls
- Ensure error messages are correctly assigned to Throwable.message
- Adjust safeApiCall to properly propagate error messages
- Simplifies using `fold`
- Handle cases more concisely
- Log exceptions only in `onFailure` block
abstracts retrofit and okhttp builders
adds ROOM as persistence for Users
migrates source of truth to local datasource
adds internet connection responsiveness
adds auto-sync
adds image for case user list is empty and no internet
extracts contact screen title from list for persistence
wagarcdev and others added 30 commits August 27, 2024 20:04
# Conflicts:
#	feature/contacts/src/test/kotlin/com/picpay/desafio/android/feature/contacts/viewmodel/ContactsScreenViewModelTest.kt
# Conflicts:
#	feature/contacts/src/test/kotlin/com/picpay/desafio/android/feature/contacts/viewmodel/ContactsScreenViewModelTest.kt
desafio_2024
abstracts LocalUserFlowUseCase into interface and impl
adds FakeLocalUsersFlowUseCase
adds extra type dependency to contacts module test
desafio_2024
# Conflicts:
#	gradle.properties
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant