Skip to content

Comments

[T-8323][17.0][IMP] base_external_api: improvements#37

Merged
Sygel-bot merged 5 commits into17.0from
17.0-T-8323
Oct 15, 2025
Merged

[T-8323][17.0][IMP] base_external_api: improvements#37
Sygel-bot merged 5 commits into17.0from
17.0-T-8323

Conversation

@Tisho99
Copy link
Contributor

@Tisho99 Tisho99 commented Sep 26, 2025

No description provided.

@Tisho99 Tisho99 changed the title [T-8323][FIX] base_external_api: let all users's action trigger external API calls [T-8323][17.0][FIX] base_external_api: let all users's action trigger external API calls Sep 26, 2025
@codecov
Copy link

codecov bot commented Sep 26, 2025

Codecov Report

❌ Patch coverage is 97.08029% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
base_external_api/models/external_api_config.py 88.88% 3 Missing and 1 partial ⚠️
Files with missing lines Coverage Δ
base_external_api/models/external_api_log.py 100.00% <100.00%> (ø)
base_external_api/tests/test_base_external_api.py 100.00% <100.00%> (ø)
base_external_api/models/external_api_config.py 87.09% <88.88%> (+3.56%) ⬆️
🚀 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.

@Tisho99 Tisho99 force-pushed the 17.0-T-8323 branch 2 times, most recently from a177678 to 1b9849f Compare October 9, 2025 16:43
@Tisho99 Tisho99 changed the title [T-8323][17.0][FIX] base_external_api: let all users's action trigger external API calls [T-8323][17.0][FIX] base_external_api: improvements Oct 9, 2025
@Tisho99 Tisho99 changed the title [T-8323][17.0][FIX] base_external_api: improvements [T-8323][17.0][IMP] base_external_api: improvements Oct 9, 2025
@ValentinVinagre
Copy link
Contributor

/sygelbot merge minor

@Sygel-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 17.0-ocabot-merge-pr-37-by-ValentinVinagre-bump-minor, awaiting test results.

@Sygel-bot Sygel-bot merged commit eac0ca2 into 17.0 Oct 15, 2025
5 checks passed
@Sygel-bot
Copy link
Contributor

Congratulations, your PR was merged at 8edec71. Thanks a lot for contributing to sygel-technology. ❤️

@Sygel-bot Sygel-bot deleted the 17.0-T-8323 branch October 15, 2025 07:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants