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: fixed qa bug #5554

Merged

Conversation

skdud4659
Copy link
Member

To Reviewers

  • Minor change, self-reviewed (style, chore, ci, test, docs)
  • Previously reviewed in feature branch, no further review needed
  • Self-merge allowed for solo developers or urgent changes

Description (optional)

SSIA

Things to Talk About (optional)

Signed-off-by: NaYeong,Kim <[email protected]>
@skdud4659 skdud4659 added the self_approved/review Pull Request has been reviewed and approved by the author without requiring additional reviewers. label Jan 17, 2025
Copy link

vercel bot commented Jan 17, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
console ⬜️ Ignored (Inspect) Visit Preview Jan 17, 2025 9:32am
dashboard ⬜️ Ignored (Inspect) Visit Preview Jan 17, 2025 9:32am

@skdud4659 skdud4659 merged commit 28645a4 into cloudforet-io:feature-project-alert-manager Jan 17, 2025
8 checks passed
@skdud4659 skdud4659 deleted the feature/minor branch January 17, 2025 09:42
skdud4659 added a commit that referenced this pull request Jan 20, 2025
* ci: add check for PR author organization membership and assign them (#5521)

* chore: for ci test

Signed-off-by: Wanjin Noh <[email protected]>

* ci: add check for PR author organization membership and assign them

Signed-off-by: Wanjin Noh <[email protected]>

* ci: update pull request review labeler to handle reopened events

Signed-off-by: Wanjin Noh <[email protected]>

* ci: enhance pull request review labeler to handle missing review state

Signed-off-by: Wanjin Noh <[email protected]>

---------

Signed-off-by: Wanjin Noh <[email protected]>

* chore: ci test (#5522)

Signed-off-by: Wanjin Noh <[email protected]>

* fix(reference-store): correct logic for selecting fetcher based on version (#5536)

- Inverted the condition for determining the collector and region fetchers
- Ensured correct API client is used based on alert manager version

Signed-off-by: sulmo <[email protected]>

* chore: changed css (#5537)

Signed-off-by: NaYeong,Kim <[email protected]>

* feat: apply qa and additional requests (#5552)

* chore: changed icon color

Signed-off-by: NaYeong,Kim <[email protected]>

* feat: create alert creation button at service page

Signed-off-by: NaYeong,Kim <[email protected]>

* feat: update history error modal

Signed-off-by: NaYeong,Kim <[email protected]>

* feat: create app reference store and apply app name at timeline

Signed-off-by: NaYeong,Kim <[email protected]>

* feat: apply resource link at alert detail page

Signed-off-by: NaYeong,Kim <[email protected]>

* feat: apply 'user group' text at member modal

Signed-off-by: NaYeong,Kim <[email protected]>

* feat: apply raw data modal at webhook error tab

Signed-off-by: NaYeong,Kim <[email protected]>

* fix: fixed timeline bug

Signed-off-by: NaYeong,Kim <[email protected]>

* feat: apply filter retention

Signed-off-by: NaYeong,Kim <[email protected]>

* chore: changed translations and style

Signed-off-by: NaYeong,Kim <[email protected]>

* feat: changed create alert button position

Signed-off-by: NaYeong,Kim <[email protected]>

* feat: fixed condition form bug and translate operator

Signed-off-by: NaYeong,Kim <[email protected]>

* chore: translations

Signed-off-by: NaYeong,Kim <[email protected]>

---------

Signed-off-by: NaYeong,Kim <[email protected]>

* fix: fixed qa bug (#5554)

* fix: fixed minor bug (timeline)

Signed-off-by: NaYeong,Kim <[email protected]>

* fix: fixed qa bug

Signed-off-by: NaYeong,Kim <[email protected]>

---------

Signed-off-by: NaYeong,Kim <[email protected]>

* feat: apply sort (#5555)

Signed-off-by: NaYeong,Kim <[email protected]>

---------

Signed-off-by: Wanjin Noh <[email protected]>
Signed-off-by: sulmo <[email protected]>
Signed-off-by: NaYeong,Kim <[email protected]>
Co-authored-by: WANJIN <[email protected]>
Co-authored-by: sulmo <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pass/signedoff self_approved/review Pull Request has been reviewed and approved by the author without requiring additional reviewers.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant