Skip to content

fix: fixes proto types of deployment authorization#2080

Merged
stalniy merged 2 commits intomain-sdk-nextfrom
fix/deployment-auth2
Oct 18, 2025
Merged

fix: fixes proto types of deployment authorization#2080
stalniy merged 2 commits intomain-sdk-nextfrom
fix/deployment-auth2

Conversation

@stalniy
Copy link
Contributor

@stalniy stalniy commented Oct 17, 2025

No description provided.

@stalniy stalniy requested a review from a team as a code owner October 17, 2025 15:27
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 17, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/deployment-auth2

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link

codecov bot commented Oct 17, 2025

Codecov Report

❌ Patch coverage is 81.25000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 45.85%. Comparing base (c971672) to head (60b01e8).
⚠️ Report is 1 commits behind head on main-sdk-next.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...b/src/components/authorizations/Authorizations.tsx 0.00% 1 Missing ⚠️
...auto-top-up-message/auto-top-up-message.service.ts 0.00% 1 Missing ⚠️
...pps/deploy-web/src/utils/TransactionMessageData.ts 0.00% 1 Missing ⚠️
Additional details and impacted files
@@                Coverage Diff                @@
##           main-sdk-next    #2080      +/-   ##
=================================================
- Coverage          46.19%   45.85%   -0.34%     
=================================================
  Files               1014     1004      -10     
  Lines              28449    28109     -340     
  Branches            7404     7337      -67     
=================================================
- Hits               13141    12889     -252     
+ Misses             14202    14124      -78     
+ Partials            1106     1096      -10     
Flag Coverage Δ *Carryforward flag
api 82.26% <ø> (ø) Carriedforward from fc2bec5
deploy-web 24.32% <81.25%> (+0.04%) ⬆️
log-collector ?
notifications 88.11% <ø> (ø) Carriedforward from fc2bec5
provider-console 81.48% <ø> (ø) Carriedforward from fc2bec5
provider-proxy 84.61% <ø> (ø) Carriedforward from fc2bec5

*This pull request uses carry forward flags. Click here to find out more.

Files with missing lines Coverage Δ
apps/deploy-web/src/utils/customRegistry.ts 100.00% <100.00%> (ø)
...b/src/components/authorizations/Authorizations.tsx 0.00% <0.00%> (ø)
...auto-top-up-message/auto-top-up-message.service.ts 0.00% <0.00%> (ø)
...pps/deploy-web/src/utils/TransactionMessageData.ts 30.30% <0.00%> (+1.73%) ⬆️

... and 11 files with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

baktun14
baktun14 previously approved these changes Oct 17, 2025
@stalniy stalniy merged commit afe7d6f into main-sdk-next Oct 18, 2025
62 checks passed
@stalniy stalniy deleted the fix/deployment-auth2 branch October 18, 2025 00:51
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

Comments