Skip to content

[DSIP-62][Http Alert Plugin] Refactor http alert plugin#16484

Merged
caishunfeng merged 5 commits intoapache:devfrom
SbloodyS:dsip_62
Aug 19, 2024
Merged

[DSIP-62][Http Alert Plugin] Refactor http alert plugin#16484
caishunfeng merged 5 commits intoapache:devfrom
SbloodyS:dsip_62

Conversation

@SbloodyS
Copy link
Copy Markdown
Member

Purpose of the pull request

close #16469

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.

(or)

This pull request is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(or)

Pull Request Notice

Pull Request Notice

If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

@SbloodyS SbloodyS added UI ui and front end related improvement make more easy to user or prompt friendly backend labels Aug 19, 2024
@SbloodyS SbloodyS added this to the 3.3.0 milestone Aug 19, 2024
@SbloodyS SbloodyS self-assigned this Aug 19, 2024
@sonarqubecloud
Copy link
Copy Markdown

@SbloodyS SbloodyS requested a review from ruanwenjun August 19, 2024 06:22
Copy link
Copy Markdown
Member

@ruanwenjun ruanwenjun left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@caishunfeng caishunfeng left a comment

Choose a reason for hiding this comment

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

LGTM overall

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend document DSIP improvement make more easy to user or prompt friendly test UI ui and front end related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[DSIP-62][Http Alert Plugin] Refactor http alert plugin

3 participants