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

🚀[Release v2.14.0] Merge into Main #306

Open
wants to merge 67 commits into
base: main
Choose a base branch
from
Open

Conversation

kcw-grunt
Copy link
Contributor

@kcw-grunt kcw-grunt commented Dec 21, 2024

Overview

This is a quick release to get the recent bugfixes into production to increase stability

APOLOGIES @josikie and @andhikayuana . I added many garbage commits when trying the new CircleCI config.
I owe you some Martabak manis! 🍰

Fixes

Improvements

  • Added Arabic!
  • feat: implement basic AppsFlyer
  • Added tests BRConstants MockK+JUnit4

kcw-grunt and others added 30 commits October 7, 2024 18:10
* replaced the control in PeerManager.c

Signed-off-by: kcw-grunt <[email protected]>

* reverted BRBitcoinAmount

Signed-off-by: kcw-grunt <[email protected]>

* Cleaning up the files

Signed-off-by: kcw-grunt <[email protected]>

* Removed Pusher remnants

Signed-off-by: kcw-grunt <[email protected]>

---------

Signed-off-by: kcw-grunt <[email protected]>
* Added fiat feature based on language user chose

* added fcm for push notifications

* Update PushNotificationService.kt

* Added new Icon, removed duplicate fcm library, added in app messaging

* Added new Icon, removed duplicate fcm library, added in app messaging
* Recent newline

Signed-off-by: kcw-grunt <[email protected]>

* AF added

Refactored cruft

- AF working

Signed-off-by: kcw-grunt <[email protected]>

---------

Signed-off-by: kcw-grunt <[email protected]>
* Tech debt/add af sdk (#248)

- AF working
- Changed requiredActivity
- Added analytics error report
- test this.Activity is null or not
- Bugfix - Phrase Reminder crash
- added an exception handler for UserNotAuthenticatedException.
		 - note: this should allow for the system to display the native authorization UI when needed
		 - fixes issue - https://console.firebase.google.com/u/0/project/litewallet-beta/crashlytics/app/android:com.loafwallet/issues/09dac17241309f0e823ef597a9a82cd4
- Added dev note
- remove calls to BREventManager
- removed BREventManager
- renamed error message
- fixed Firebase Analytics event error

Signed-off-by: kcw-grunt <[email protected]>

* version bump

update gitignore

Signed-off-by: kcw-grunt <[email protected]>

---------

Signed-off-by: kcw-grunt <[email protected]>
* removed unused BRSharedPreferences

Signed-off-by: kcw-grunt <[email protected]>

* started commenting out more Timber

Signed-off-by: kcw-grunt <[email protected]>

* Added more shared preferences for syncing

Signed-off-by: kcw-grunt <[email protected]>

* Commented out many verbose Timber logs

Signed-off-by: kcw-grunt <[email protected]>

* WIP

Still trying to figure out the eplased time and last and start timestamps…???
I dunno

Signed-off-by: kcw-grunt <[email protected]>

* Moved sync measurment into the method calls

Signed-off-by: kcw-grunt <[email protected]>

* Debugged the install issue with send

- Moved all the measurements in new methods
- recalculated measure points
- Added analytics
- event did_complete_sync was shown!
- adding a dummy file to make sure file system is set
||||WIP still verifying the JSON is present
- added a blockheight label
-  changed the sleep time (a hack to allow the device time to refresh the view) to 100 ms from 500ms!!
-WIP Why is the bundle not showing on time
Signed-off-by: kcw-grunt <[email protected]>

* replaces afID to first location

Signed-off-by: kcw-grunt <[email protected]>

* Removed redundant sync measurements

Signed-off-by: kcw-grunt <[email protected]>

* code bump

Signed-off-by: kcw-grunt <[email protected]>

---------

Signed-off-by: kcw-grunt <[email protected]>
* Removed non english seed words

-Refactor tests
- Reset AnalyticsTests

Signed-off-by: kcw-grunt <[email protected]>

* Rename Bitcoinletter to Litecoinletters

-added tests
testLitecoinSymbolConstants
testAppExternalURLConstants
testFirebaseAnalyticsConstants

Signed-off-by: kcw-grunt <[email protected]>

---------

Signed-off-by: kcw-grunt <[email protected]>
…Phrase, so we just wrap with runCatching for now to avoid crash (#263)
* fix: [WIP][#265] work in progress add null checking, migrate viewpager to viewpager2 and more

* fix: [#265] finalize migration some deprecated class and implement null checking at FragmentTransactionItem

* fix: dix dismiss outside FragmentTransactionDetails
* 🚀[Release v2.11.1 071024] Merge into Main (#246)

* replaced the control in PeerManager.c

Signed-off-by: kcw-grunt <[email protected]>

* reverted BRBitcoinAmount

Signed-off-by: kcw-grunt <[email protected]>

* Cleaning up the files

Signed-off-by: kcw-grunt <[email protected]>

* Removed Pusher remnants

Signed-off-by: kcw-grunt <[email protected]>

---------

Signed-off-by: kcw-grunt <[email protected]>

* version bump

Signed-off-by: kcw-grunt <[email protected]>

* Removed donation button

removed xml
removed fragments
removed references of the donation
removed analytical events

Signed-off-by: kcw-grunt <[email protected]>

* fix: prevent activity close

* code bump

Signed-off-by: kcw-grunt <[email protected]>

* enabled for Debug

Signed-off-by: kcw-grunt <[email protected]>

* fix: [#264] add null checking for BRKeyStore.removeAliasAndFiles (#270)

* fix: [#265] add null checking, migrate viewpage… (#271)

* fix: [WIP][#265] work in progress add null checking, migrate viewpager to viewpager2 and more

* fix: [#265] finalize migration some deprecated class and implement null checking at FragmentTransactionItem

* fix: dix dismiss outside FragmentTransactionDetails

* code bump

Signed-off-by: kcw-grunt <[email protected]>

* version bump

Signed-off-by: kcw-grunt <[email protected]>

---------

Signed-off-by: kcw-grunt <[email protected]>
Co-authored-by: andhikayuana <[email protected]>
…led multiple times at onBindViewHolder (#277)

* fix: [#258] avoid OOM with cache the result of opsAll, previously called multiple times at onBindViewHolder

* chore: [#258] cleanup comment
…tCurrency can produce null (#278)

* fix: [#266] add null checking and default value based on iOS since getCurrency can produce null

* chore: [#266] cleanup comment
…mockk (#282)

* chore: [#274] initial setup test using JUnit & MockK

* chore: [#274] add more test case at LocaleHelperTest

* fix: [#274] fix default language from shared preferences

* fix: [#274] disable language split on AAB

* chore: [#274] rename method
* Check reenter pin freeze after the auth prompt deleted.

Signed-off-by: kcw-grunt <[email protected]>

* cleaned up deletion

Signed-off-by: kcw-grunt <[email protected]>

* Many removals

- Removed the fingerprint xml
- Removed the Fingerprint Fragment and java files
- Removed the enum referencing the fingerprint prompt
- Removed check of fingerprint enrollment and checks
- Removed biometrics and fingerprint permissions

Signed-off-by: kcw-grunt <[email protected]>

* removed fingerprint image

code and version bump

Signed-off-by: kcw-grunt <[email protected]>

* chore: remove FragmentFingerprint.java since, not used and failed to build

---------

Signed-off-by: kcw-grunt <[email protected]>
Co-authored-by: josikie <[email protected]>
Co-authored-by: andhikayuana <[email protected]>
* 🚀[Release v2.11.1 071024] Merge into Main (#246)

* replaced the control in PeerManager.c

Signed-off-by: kcw-grunt <[email protected]>

* reverted BRBitcoinAmount

Signed-off-by: kcw-grunt <[email protected]>

* Cleaning up the files

Signed-off-by: kcw-grunt <[email protected]>

* Removed Pusher remnants

Signed-off-by: kcw-grunt <[email protected]>

---------

Signed-off-by: kcw-grunt <[email protected]>

* 🚀[Release v2.12.2 20241118] Merge into Main (#273)

* 🚀[Release v2.11.1 071024] Merge into Develop (#245)

* replaced the control in PeerManager.c

Signed-off-by: kcw-grunt <[email protected]>

* reverted BRBitcoinAmount

Signed-off-by: kcw-grunt <[email protected]>

* Cleaning up the files

Signed-off-by: kcw-grunt <[email protected]>

* Removed Pusher remnants

Signed-off-by: kcw-grunt <[email protected]>

---------

Signed-off-by: kcw-grunt <[email protected]>

* Added Fiat feature, and FCM for push notifications (#247)

* Added fiat feature based on language user chose

* added fcm for push notifications

* Update PushNotificationService.kt

* Added new Icon, removed duplicate fcm library, added in app messaging

* Added new Icon, removed duplicate fcm library, added in app messaging

* Tech debt/add af sdk (#248)

* Recent newline

Signed-off-by: kcw-grunt <[email protected]>

* AF added

Refactored cruft

- AF working

Signed-off-by: kcw-grunt <[email protected]>

---------

Signed-off-by: kcw-grunt <[email protected]>

* Techdebt/refactor brevents syncmarkers (#254)

* Tech debt/add af sdk (#248)

- AF working
- Changed requiredActivity
- Added analytics error report
- test this.Activity is null or not
- Bugfix - Phrase Reminder crash
- added an exception handler for UserNotAuthenticatedException.
		 - note: this should allow for the system to display the native authorization UI when needed
		 - fixes issue - https://console.firebase.google.com/u/0/project/litewallet-beta/crashlytics/app/android:com.loafwallet/issues/09dac17241309f0e823ef597a9a82cd4
- Added dev note
- remove calls to BREventManager
- removed BREventManager
- renamed error message
- fixed Firebase Analytics event error

Signed-off-by: kcw-grunt <[email protected]>

* version bump

update gitignore

Signed-off-by: kcw-grunt <[email protected]>

---------

Signed-off-by: kcw-grunt <[email protected]>

* Techdebt/ fixed send issue add syncing measurements (#255)

* removed unused BRSharedPreferences

Signed-off-by: kcw-grunt <[email protected]>

* started commenting out more Timber

Signed-off-by: kcw-grunt <[email protected]>

* Added more shared preferences for syncing

Signed-off-by: kcw-grunt <[email protected]>

* Commented out many verbose Timber logs

Signed-off-by: kcw-grunt <[email protected]>

* WIP

Still trying to figure out the eplased time and last and start timestamps…???
I dunno

Signed-off-by: kcw-grunt <[email protected]>

* Moved sync measurment into the method calls

Signed-off-by: kcw-grunt <[email protected]>

* Debugged the install issue with send

- Moved all the measurements in new methods
- recalculated measure points
- Added analytics
- event did_complete_sync was shown!
- adding a dummy file to make sure file system is set
||||WIP still verifying the JSON is present
- added a blockheight label
-  changed the sleep time (a hack to allow the device time to refresh the view) to 100 ms from 500ms!!
-WIP Why is the bundle not showing on time
Signed-off-by: kcw-grunt <[email protected]>

* replaces afID to first location

Signed-off-by: kcw-grunt <[email protected]>

* Removed redundant sync measurements

Signed-off-by: kcw-grunt <[email protected]>

* code bump

Signed-off-by: kcw-grunt <[email protected]>

---------

Signed-off-by: kcw-grunt <[email protected]>

* Techdebt/test refactor cruft removal (#250)

* Removed non english seed words

-Refactor tests
- Reset AnalyticsTests

Signed-off-by: kcw-grunt <[email protected]>

* Rename Bitcoinletter to Litecoinletters

-added tests
testLitecoinSymbolConstants
testAppExternalURLConstants
testFirebaseAnalyticsConstants

Signed-off-by: kcw-grunt <[email protected]>

---------

Signed-off-by: kcw-grunt <[email protected]>

* fix: [#152] make sure using Fragment, FragmentManager and FragmentTransaction from androidx (#260)

* fix: [#126] the issue came from FragmentBalanceSeedReminder.fetchSeedPhrase, so we just wrap with runCatching for now to avoid crash (#263)

* version bump

Signed-off-by: kcw-grunt <[email protected]>

* Removed donation button

removed xml
removed fragments
removed references of the donation
removed analytical events

Signed-off-by: kcw-grunt <[email protected]>

* fix: prevent activity close

* code bump

Signed-off-by: kcw-grunt <[email protected]>

* enabled for Debug

Signed-off-by: kcw-grunt <[email protected]>

* fix: [#264] add null checking for BRKeyStore.removeAliasAndFiles (#270)

* fix: [#265] add null checking, migrate viewpage… (#271)

* fix: [WIP][#265] work in progress add null checking, migrate viewpager to viewpager2 and more

* fix: [#265] finalize migration some deprecated class and implement null checking at FragmentTransactionItem

* fix: dix dismiss outside FragmentTransactionDetails

* code bump

Signed-off-by: kcw-grunt <[email protected]>

* version bump

Signed-off-by: kcw-grunt <[email protected]>

---------

Signed-off-by: kcw-grunt <[email protected]>
Co-authored-by: Josi Kie <[email protected]>
Co-authored-by: Andhika Yuana <[email protected]>

* bumped version and code

* compiled the env (#284)

please check @andhikayuana @josikie  that you are able to compile

* Users complained that recyclerView was setting to the wrong language

- Polished to index to a language
- bump code

* chore: [#126] simplify exception handling at BRKeyStore._getData and record exception to Firebase Crashlytics (#283)

---------

Signed-off-by: kcw-grunt <[email protected]>
Co-authored-by: Josi Kie <[email protected]>
Co-authored-by: Andhika Yuana <[email protected]>
)

* Added abilities to copy transaction id by click the transaction id

* Update build.gradle
* chore: [test] add BRConstantsTest for unitTest and deprecated ConstantsTests inside androidTest

* chore: [test] remove unused test files

* chore: [test] add todo to fill the tests and change package name

* chore: [test] add basic example

* chore: [test] integrate with ci/cd

* chore: [test][ci] fix Cannot find a definition for command named android/restore-gradle-cache

* chore: [test][ci] fix Cannot find a definition for command named android/run-tests

* chore: [test][ci] fix Unexpected argument(s): test-command

* chore: [test][ci] fix Cannot find a definition for executor named android/android-docker

* chore: [test][ci] add default for gradle.properties

* chore: [test][ci] avoid breaking in build.gradle

* chore: [test][ci] enable androidx and jetifier

* chore: [test][ci] init submodule

* chore: [test][ci] just using run command for execute unit test step

* chore: [test][ci] add env for google-services.json

* chore: [test][ci] add env for google-services.json

* chore: [test][ci] add env for google-services.json

* Major refactor of code to move classes to the ideal location

- Removed cruft
- Moved all tests to the unitTest directory
- Remove poorly placed tests into (AndroidTest)
- Commented  old code into the new Kotlin class(es)
- working in the first test of currency
- Removed the paymentRequest method
- Renamed Classes

* chore: resolve CurrencyTests

* Added notes

---------

Co-authored-by: Kerry Washington <[email protected]>
* feat: [remote-config] wip for remote config feature toggle

* feat: [remote-config] implement base url toggle for prod and dev

* feat: [test] add BRApiManagerTest

* fix: fix getBaseUrlProd for FragmentBuy and add test

* chore: change BRApiManager.fetchRates using getBaseUrlProd
* feat: add notification channel

* feat: add PermissionUtil

* feat: request notification permission for API 33 or above

* feat: implement firebase in-app messaging

* feat: implement simple cloud messaging

* feat: add NotificationHandlerTest
* fix: fix close button at activity_display_currency.xml

* fix: fix display currency
* WIP circleCI

* Create GoogleServicesTests.kt

WIP
Need to add koltin json parser

* WIP

Trying to mock testing of the key objects in the JSON
Adding the GoogleService Tests

* removed unnecessary dependency

* chore: update specific android docker image version to 2024.11.1

* chore: update image using 2024.07.1-ndk

* chore: export GOOGLE_SERVICES_JSON to $BASH_ENV first before decode to file

---------

Co-authored-by: andhikayuana <[email protected]>
* chore: remove forgot_seed_phrase_or_pin_text text at activity_pin.xml

* fix: [#288] catch all exception and return null

* chore: wip keystore related

* chore: wip keystore related for CipherBox

* chore: wip keystore related for CipherBox

* feat: add new CipherBox, CipherStorage, KeyStoreKeyGenerator and KeyStoreManager to handle keystore related

* feat: add keyStoreManager to BreadApp and change _getData method at BRKeyStore

* Revert "chore: remove forgot_seed_phrase_or_pin_text text at activity_pin.xml"

This reverts commit 04b02ac.

* chore: resolve conflict and implement toggle for new KeyStoreManager
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
android Android related only 🚀 release Related to a production release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants