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.

Sign up for GitHub

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

Jump to bottom

Upgrade Client Core Library #871

Merged
merged 78 commits into from
Nov 4, 2022
Merged

Upgrade Client Core Library #871

merged 78 commits into from
Nov 4, 2022

Conversation

ndegwamartin
Copy link
Contributor

@ndegwamartin ndegwamartin commented May 26, 2022 •
edited
Loading

Changes This PR makes

  • Migrate core dependencies to latest versions
  • Migrates Crashlytics from Fabric to Firebase
  • Cleans up unused SQLCipher artefacts
  • Reverts the MLS implementation for Location Picker. Now tracked under Multi Language Support for LocationPickerView Entries #870
  • Refactor Image handling implementation - use Glide library
  • Remove Apache HTTP Client to HTTPURLConnection
  • Clean up unused Helper Classes and Static Libraries
  • Implement OAuth Credentials Error Handling
  • Bump up artifact version to 6.0.0
  • Fix previously disabled/ignored unit test cases
  • Remove/Refactor various AsyncTasks
  • Implement configuration to Disable images fetch
  • Fix user logged out immediately after offline login: Closes Refactor re-authentication implementation to enhance UX #787

@ndegwamartin ndegwamartin requested review from qiarie and ekigamba May 26, 2022 12:51
ndegwamartin and others added 18 commits May 30, 2022 15:37
@ndegwamartin
Client core lib artefact release automation CI/CD 👷
ed97ea6
@ndegwamartin
Update CI configuration file 🔧
b10c0ff
@hamza-vd @ndegwamartin
Fix Password Reset Webpage Error
d348898
@hamza-vd @ndegwamartin
Bump up core version
82edb51
@hamza-vd @ndegwamartin
Reformat Code
29ae8fa
@ndegwamartin
Migrate crashlytics library
55aa9c7 
- Migrate from Fabric to Firebase crashlytics
- Refactor - move ECDetailsTest class
@ndegwamartin
Update dependent libraries
75b6cb8
@ndegwamartin
Remove deprecated sqlcipher binaries
422875c
@ndegwamartin
Revert Location translations
244bdbe 
- Revert the translation locations #870
@ndegwamartin
5.0.4 PREVIEW SNAPSHOT
7b317cb
@ndegwamartin
Migrate Jackson annotation library
fd66483
@ndegwamartin
Remove redundant static libraries
d8daf69
@ndegwamartin
Refactor Image handling implementation
8f8cf2c 
- Refactor custom Volley implementation use Glide library
@ndegwamartin
Refactor Client HTTP Requests Implementaton
b37883c 
- Remove deprecated Apache client, switch to HTTPURLConnection
- Refactor download files implementation to use HTTPURLConnection
- Update status code constant refs to HTTPURLConnection enum
@ndegwamartin
Clean up Unused Helper Classes
a944881
@ndegwamartin
Implement OAuth Credentials Error Handling
7d787e2
@ndegwamartin
Migrate Dependencies To Latest
20240d8 
- Migrate all project dependencies to use newest versions
- Clean up code
@ndegwamartin
Bump up artifact version to 6.0.0
e2bdf41
@ndegwamartin ndegwamartin force-pushed the migrate-crashlytics branch from e7e2f41 to e2bdf41 Compare May 30, 2022 16:37
@ndegwamartin ndegwamartin changed the title Migrate Crashlytics to Firebase Upgrade Client Core Library May 30, 2022
@ndegwamartin
version bump
fc3a002
@ndegwamartin
Upgradle gradle build tools
aad868c 
Upgrade gradle configs
@ndegwamartin
Upgrade testing framework
58d27d1 
- Replace Power mock
- Refactor fix failing Unit Tests
- Fix failing build
@ndegwamartin
Refactor/Fix failing unit tests
413c4fb
@ndegwamartin
Refactor Authentication Implementation/Fix Unit Tests
9987b9b 
- Refactor login implementation rememove deprecated AsyncTasks
- Fix Authentication unit tests post dependency migration
@ndegwamartin
Refactor/Fix failing Unit Tests ✅
126332b 
- Refactor tests after dependency upgrades
- Refactor/Migrate Powermock to Mockito unit tests
@ndegwamartin
Merge branch 'master' into migrate-crashlytics
546268c
@ndegwamartin
Version 6.0.0. ALPHA
32dcaa4
@ndegwamartin
Copy link
Contributor Author

REALEASE NOTES

ndegwamartin and others added 18 commits September 20, 2022 09:22
@ndegwamartin
Code clean up/Refactor
aea5366 
Refactor unit tests
Merge branch 'master' into migrate-crashlytics
@ndegwamartin
Refactor for testability ✅
2c94dec
@SebaMutuku
Addition of user details to shared Prefs
3dda4b3
@SebaMutuku
Addition of user details to shared Prefs
d041742
@SebaMutuku
saveProviderBaseEntityIdOnSharedPref
b0f0ee9
@SebaMutuku
saveProviderBaseEntityIdOnSharedPref
b78c9d4
@SebaMutuku
Codacy updates
98031b7
@SebaMutuku
Codacy updates
2b450bc
@junaidwarsivd
issue 906 added translated object parsing
1284c31
@junaidwarsivd
updated the test for extract TranslatableValue
b6809fb
@junaidwarsivd
removed the unnessary Assert ref
987c3bb
@junaidwarsivd
Merge branch 'migrate-crashlytics' of https://github.com/opensrp/open…
b474095 
…srp-client-core into migrate-crashlytics
@junaidwarsivd
Merge branch 'migrate-crashlytics' into anc-issue-fix-906
06276b8
@junaidwarsivd
added fixes for the extended radio buttons not addeing humab readable…
1067621 
… values
@ndegwamartin
Merge remote-tracking branch 'origin/master' into migrate-crashlytics
2974f25
@junaidwarsivd
Merge remote-tracking branch 'origin/add_user_details_to_shared_prefs…
9dc6eb6 
…' into anc-issue-fix-906

# Conflicts:
#	gradle.properties
@junaidwarsivd
Merge remote-tracking branch 'origin/test_add_user_details_on_sharedp…
bedfc97 
…refs' into anc-issue-fix-906
@dubdabasoduba
Merge pull request #892 from opensrp/anc-issue-fix-906
5fdb3dd 
Anc issue fix 906 for the human readable values not updating in events when the translation is on
@SebaMutuku SebaMutuku mentioned this pull request Oct 26, 2022
Merged
ekigamba
ekigamba previously approved these changes Oct 27, 2022
View reviewed changes
@ekigamba ekigamba mentioned this pull request Nov 1, 2022
Open
4 tasks
@ekigamba
Copy link
Contributor

ekigamba commented Nov 1, 2022

Issue to follow-up on coverage drop #894

@ndegwamartin
Bump up artifact version
3a6bf41 
Update dependencies
@ekigamba ekigamba self-requested a review November 4, 2022 09:20
@ndegwamartin ndegwamartin merged commit abbd694 into master Nov 4, 2022
@ndegwamartin ndegwamartin deleted the migrate-crashlytics branch November 4, 2022 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Rkareko Rkareko Rkareko left review comments

@hilpitome hilpitome hilpitome left review comments

@ekigamba ekigamba ekigamba approved these changes

@qiarie qiarie Awaiting requested review from qiarie

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Refactor re-authentication implementation to enhance UX
10 participants
@ndegwamartin @ekigamba @hilpitome @Rkareko @hamza-vd @qiarie @allan-on @SebaMutuku @junaidwarsivd @dubdabasoduba