Skip to content

CustomSettings Implementation#2949

Open
keval-finimble wants to merge 21 commits into
masterfrom
2758-feature-customisation-json
Open

CustomSettings Implementation#2949
keval-finimble wants to merge 21 commits into
masterfrom
2758-feature-customisation-json

Conversation

@keval-finimble

Copy link
Copy Markdown
Contributor
  • Enhanced the cache functionality by changing the return type of lockedTokens in CustomSettings.
  • Added other fields in the json and call them directly from the json file in CustomSettings.

Keval Shukla added 2 commits November 15, 2022 16:54
…ectly from Activity and Fragments.

- Enhanced the cache functionality by changing the return type of lockedTokens in CustomSettings.
- Added other fields in the json and call them directly from the json file in CustomSettings.
…ectly from Activity and Fragments.

- Enhanced the cache functionality by changing the return type of lockedTokens in CustomSettings.
- Added other fields in the json and call them directly from the json file in CustomSettings.
@github-actions

Copy link
Copy Markdown

Pull reviewers stats

Stats of the last 30 days for alpha-wallet-android:

User Total reviews Time to review Total comments
seabornlee
🥇
14
▀▀▀▀
3h 36m
10
▀▀▀▀▀▀▀▀
JamesSmartCell
🥈
11
▀▀▀
1d 41m
▀▀▀▀
2
▀▀
justindg
🥉
8
▀▀
1d 5h 2m
▀▀▀▀▀
1

Comment thread app/src/main/java/com/alphawallet/app/di/RepositoriesModule.java Outdated
Comment thread app/src/main/java/com/alphawallet/app/ui/widget/adapter/TokensAdapter.java Outdated
Comment thread app/src/main/java/com/alphawallet/app/ui/SellDetailActivity.java Outdated
@codecov

codecov Bot commented Nov 16, 2022

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 8.08301% with 1683 lines in your changes missing coverage. Please review.
✅ Project coverage is 7.68%. Comparing base (7784baa) to head (41c7a3e).
⚠️ Report is 263 commits behind head on master.

Files with missing lines Patch % Lines
...allet/app/walletconnect/AWWalletConnectClient.java 6.93% 161 Missing ⚠️
...om/alphawallet/app/ui/WalletConnectV2Activity.java 0.00% 160 Missing ⚠️
...hawallet/app/repository/TransactionRepository.java 6.14% 107 Missing ⚠️
...va/com/alphawallet/app/service/CustomSettings.java 35.82% 82 Missing and 4 partials ⚠️
...com/alphawallet/app/entity/tokens/ERC721Token.java 0.00% 76 Missing ⚠️
...et/app/walletconnect/TransactionDialogBuilder.java 0.00% 71 Missing ⚠️
...c/main/java/com/alphawallet/app/entity/Wallet.java 0.00% 57 Missing ⚠️
.../alphawallet/app/widget/ActionSheetSignDialog.java 0.00% 57 Missing ⚠️
...tity/walletconnect/WalletConnectV2SessionItem.java 0.00% 55 Missing ⚠️
...phawallet/app/ui/WalletConnectSessionActivity.java 0.00% 49 Missing ⚠️
... and 85 more
Additional details and impacted files
@@             Coverage Diff             @@
##             master   #2949      +/-   ##
===========================================
+ Coverage      7.62%   7.68%   +0.05%     
- Complexity      836     856      +20     
===========================================
  Files           612     639      +27     
  Lines         41863   42918    +1055     
  Branches       4226    4310      +84     
===========================================
+ Hits           3194    3300     +106     
- Misses        38395   39346     +951     
+ Partials        274     272       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Keval Shukla added 8 commits November 16, 2022 19:35
…ed a new function to load lockedChains, lockedTokens and exclusiveChains.
…ectly from Activity and Fragments.

- Enhanced the cache functionality by changing the return type of lockedTokens in CustomSettings.
- Added other fields in the json and call them directly from the json file in CustomSettings.
…ectly from Activity and Fragments.

- Enhanced the cache functionality by changing the return type of lockedTokens in CustomSettings.
- Added other fields in the json and call them directly from the json file in CustomSettings.
…ed a new function to load lockedChains, lockedTokens and exclusiveChains.
… into 2758-feature-customisation-json

# Conflicts:
#	app/src/main/java/com/alphawallet/app/service/PriceAlertsService.java
#	app/src/main/java/com/alphawallet/app/viewmodel/WalletViewModel.java
Comment thread app/src/androidTest/java/com/alphawallet/app/CustomSettingsTest.java Outdated
Comment thread app/src/androidTest/java/com/alphawallet/app/CustomSettingsTest.java Outdated
Keval Shukla added 10 commits November 22, 2022 19:52
…ectly from Activity and Fragments.

- Enhanced the cache functionality by changing the return type of lockedTokens in CustomSettings.
- Added other fields in the json and call them directly from the json file in CustomSettings.
…ectly from Activity and Fragments.

- Enhanced the cache functionality by changing the return type of lockedTokens in CustomSettings.
- Added other fields in the json and call them directly from the json file in CustomSettings.
…ed a new function to load lockedChains, lockedTokens and exclusiveChains.
…ectly from Activity and Fragments.

- Enhanced the cache functionality by changing the return type of lockedTokens in CustomSettings.
- Added other fields in the json and call them directly from the json file in CustomSettings.
…ed a new function to load lockedChains, lockedTokens and exclusiveChains.
… into 2758-feature-customisation-json

# Conflicts:
#	app/src/main/java/com/alphawallet/app/service/PriceAlertsService.java
#	app/src/main/java/com/alphawallet/app/ui/AssetDisplayActivity.java
#	app/src/main/java/com/alphawallet/app/ui/HomeActivity.java
#	app/src/main/java/com/alphawallet/app/ui/NFTAssetsFragment.java
#	app/src/main/java/com/alphawallet/app/ui/widget/adapter/NonFungibleTokenAdapter.java
Comment thread app/src/main/java/com/alphawallet/app/ui/widget/adapter/TokensAdapter.java Outdated
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