Skip to content

feat: Plat-6303 store solana submitted transactions to db and run status check logic#398

Merged
zeljkoX merged 71 commits into
mainfrom
plat-6303-store-sent-transactions-to-db2
Aug 8, 2025
Merged

feat: Plat-6303 store solana submitted transactions to db and run status check logic#398
zeljkoX merged 71 commits into
mainfrom
plat-6303-store-sent-transactions-to-db2

Conversation

@zeljkoX

@zeljkoX zeljkoX commented Jul 30, 2025

Copy link
Copy Markdown
Collaborator

Summary

This PR will add logic to store solana submitted transactions to repository and to run status check for those transactions.

Currently txs are submitted from rpc src/domain/relayer/solana/rpc/methods/sign_and_send_transaction.rs method.

Tx models are stored and based on them status check jobs are scheduled.

Testing Process

Checklist

  • Add a reference to related issues in the PR description.
  • Add unit tests if applicable.

@zeljkoX zeljkoX requested review from a team as code owners July 30, 2025 14:31
@zeljkoX zeljkoX changed the title feat: Plat-6303 store sent transactions to db2 feat: Plat-6303 store sent transactions to db Jul 30, 2025
@zeljkoX zeljkoX changed the base branch from plat-6681-expose-crud-api-endpoints-2 to main July 30, 2025 14:33
@zeljkoX zeljkoX changed the base branch from main to plat-6681-expose-crud-api-endpoints-2 July 30, 2025 14:33
@zeljkoX zeljkoX changed the title feat: Plat-6303 store sent transactions to db feat: Plat-6303 store solana submitted transactions to db and run status check logic Jul 30, 2025
Base automatically changed from plat-6681-expose-crud-api-endpoints-2 to main July 31, 2025 06:49
@codecov

codecov Bot commented Jul 31, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 94.89796% with 55 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.7%. Comparing base (f3c3426) to head (e861fa2).
⚠️ Report is 13 commits behind head on main.

Files with missing lines Patch % Lines
src/services/provider/solana/mod.rs 0.0% 27 Missing ⚠️
src/domain/transaction/solana/status.rs 95.7% 19 Missing ⚠️
...er/solana/rpc/methods/sign_and_send_transaction.rs 94.6% 6 Missing ⚠️
src/domain/relayer/solana/rpc/methods/utils.rs 98.8% 2 Missing ⚠️
...rc/domain/transaction/solana/solana_transaction.rs 98.5% 1 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff           @@
##            main    #398    +/-   ##
======================================
  Coverage   92.6%   92.7%            
======================================
  Files        214     215     +1     
  Lines      69159   70104   +945     
======================================
+ Hits       64081   64987   +906     
- Misses      5078    5117    +39     
Flag Coverage Δ
integration 0.5% <0.0%> (-0.1%) ⬇️
properties <0.1% <0.0%> (-0.1%) ⬇️
unittests 92.7% <94.8%> (+<0.1%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.

};

pub mod solana;
pub use solana::*;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do we need to use solana::* ? I would rather prefer to specify transaction::solana::... to limit what's exposed under transaction::

@zeljkoX zeljkoX merged commit e8420bc into main Aug 8, 2025
23 of 24 checks passed
@zeljkoX zeljkoX deleted the plat-6303-store-sent-transactions-to-db2 branch August 8, 2025 08:20
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants