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

[11.2] Improvements to confirm #987

Merged
merged 9 commits into from
Aug 21, 2024
Merged

Conversation

rez1dent3
Copy link
Member

@rez1dent3 rez1dent3 commented Aug 21, 2024

Within the transaction, make an additional query to the database to check the transaction confirmation.

@rez1dent3 rez1dent3 self-assigned this Aug 21, 2024
Copy link

github-actions bot commented Aug 21, 2024

Qodana for PHP

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Detected 3 dependencies

Third-party software list

This page lists the third-party software dependencies used in project

Dependency Version Licenses
brick/math 0.12.1 MIT
ramsey/collection 2.0.0 MIT
ramsey/uuid 4.7.6 MIT
Contact Qodana team

Contact us at qodana-support@jetbrains.com

Copy link

codecov bot commented Aug 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (7806d59) to head (7a4dc4b).
Report is 10 commits behind head on master.

Additional details and impacted files
@@              Coverage Diff              @@
##             master      #987      +/-   ##
=============================================
+ Coverage     99.86%   100.00%   +0.13%     
- Complexity      589       590       +1     
=============================================
  Files            88        88              
  Lines          2178      2182       +4     
=============================================
+ Hits           2175      2182       +7     
+ Misses            3         0       -3     

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

@rez1dent3 rez1dent3 changed the title wallets.walletTransactions [11.2] Improvements to confirm Aug 21, 2024
@rez1dent3 rez1dent3 merged commit dc15953 into master Aug 21, 2024
74 checks passed
@rez1dent3 rez1dent3 deleted the unit-wallets.walletTransactions branch August 29, 2024 17:55
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