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

[Merge] Dev -> Main #31

Merged
merged 48 commits into from
May 26, 2024
Merged

[Merge] Dev -> Main #31

merged 48 commits into from
May 26, 2024

Conversation

0xIchigo
Copy link
Collaborator

No description provided.

davidkathoh and others added 30 commits May 19, 2024 16:20
…est handler cannot Deserialise an empty response body
fix(request_handler) Handle Empty Response Body
…est handler cannot Deserialise an empty response body
davidkathoh and others added 18 commits May 24, 2024 00:23
# Conflicts:
#	tests/test_enhanced_transactions.rs
#	tests/webhook/test_append_addresses_to_webhook.rs
#	tests/webhook/test_create_webhook.rs
#	tests/webhook/test_delete_webhook.rs
#	tests/webhook/test_edit_webhook.rs
#	tests/webhook/test_get_all_webhooks.rs
#	tests/webhook/test_get_webhook_by_id.rs
#	tests/webhook/test_remove_addresses_from_webhook.rs
 test(Webhooks & EnhancedTransactions): Improved Test Coverage for Webhooks and Enhanced Transactions
…nding

feat(optimized_transactions): Add `send_smart_transaction` Functionality
feat(docs): Update Docs for Smart Transactions and `HeliusFactory`
@0xIchigo 0xIchigo merged commit b8fac69 into main May 26, 2024
2 checks passed
final_instructions.clone(),
pubkey,
config.lookup_tables.clone().unwrap_or_default(),
&config.from_keypair,

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [clippy] reported by reviewdog 🐶

warning: this expression creates a reference which is immediately dereferenced by the compiler
   --> src/optimized_transaction.rs:229:17
    |
229 |                 &config.from_keypair,
    |                 ^^^^^^^^^^^^^^^^^^^^ help: change this to: `config.from_keypair`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
    = note: `#[warn(clippy::needless_borrow)]` on by default

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.

2 participants