Skip to content

Add Solana - #821

Closed
sneurlax wants to merge 9 commits into
stagingfrom
solana
Closed

Add Solana#821
sneurlax wants to merge 9 commits into
stagingfrom
solana

Conversation

@sneurlax

@sneurlaxsneurlax commented Mar 21, 2024

Copy link
Copy Markdown
Member

It's hidden behind a toggle in the developer menu.

Thank you @detherminal for this work

We still need:

  • to use Tor with Solana's RpcClient
  • a Solana logo

Please also refer to the comments on #816, we may want to track issues for:

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 1.10294% with 269 lines in your changes are missing coverage. Please review.

Project coverage is 5.40%. Comparing base (414803d) to head (f5da852).
Report is 68 commits behind head on staging.

FilesPatch %Lines
lib/wallets/wallet/impl/solana_wallet.dart0.00%170 Missing ⚠️
lib/wallets/crypto_currency/coins/solana.dart0.00%17 Missing ⚠️
...gs_views/global_settings_view/hidden_settings.dart0.00%16 Missing ⚠️
lib/utilities/prefs.dart0.00%9 Missing ⚠️
lib/utilities/enums/coin_enum.dart27.27%8 Missing ⚠️
lib/utilities/constants.dart0.00%7 Missing ⚠️
...gs_view/manage_nodes_views/add_edit_node_view.dart0.00%6 Missing ⚠️
...ngs_view/manage_nodes_views/node_details_view.dart0.00%5 Missing ⚠️
lib/utilities/default_nodes.dart0.00%5 Missing ⚠️
lib/widgets/node_card.dart0.00%5 Missing ⚠️
... and 10 more

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@ Coverage Diff @@## staging #821 +/- ##
==========================================
- Coverage 5.83% 5.40% -0.44% 
==========================================
Files 741 780 +39 Lines 70283 75414 +5131 ==========================================
- Hits 4100 4075 -25 - Misses 66183 71339 +5156 

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

@sneurlax
sneurlax deleted the solana branch April 24, 2024 03:02
Sign up for freeto 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.

3 participants

@sneurlax@codecov-commenter@detherminal