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

fix: 修复已禁用插件仍能通过监控平台的相关订阅步骤触发下发动作问题(closed #2313) #2314

Merged
merged 1 commit into from
Dec 27, 2024

Conversation

chalice-1831
Copy link
Collaborator

@chalice-1831 chalice-1831 commented Jul 3, 2024

PR

CheckList(PR Assigners)

确认已完成以下操作:

  • 已将分支与当前合入分支同步
  • 已在关联 Issue 中补充「功能自测」部分内容
  • 已更新关联 Issue 的状态 backlog -> todo -> for test -> tested(需通过测试同学测试) -> done
  • 将 PR 添加上 pr/reviewable

CheckList(PR Reviewers)

确认已完成以下操作:

  • 检查关联 Issue 「功能自测」部分内容
  • 必要时打回 pr/reviewable,要求重新修改
  • 及时 Approve

@chalice-1831 chalice-1831 changed the base branch from v2.4.7-dev to v2.4.8-dev December 27, 2024 02:36
@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 33.33333% with 2 lines in your changes missing coverage. Please review.

Project coverage is 75.16%. Comparing base (4f86151) to head (4f0c0a2).
Report is 3 commits behind head on v2.4.8-dev.

Files with missing lines Patch % Lines
apps/backend/subscription/steps/plugin.py 33.33% 2 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff               @@
##           v2.4.8-dev    #2314      +/-   ##
==============================================
- Coverage       75.16%   75.16%   -0.01%     
==============================================
  Files             407      407              
  Lines           29544    29547       +3     
==============================================
+ Hits            22208    22210       +2     
- Misses           7336     7337       +1     

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

@wyyalt wyyalt merged commit 81b1d3c into TencentBlueKing:v2.4.8-dev Dec 27, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/reviewable 可被 review 的 PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants