Skip to content

fix(opsflow): apply latest console query#5819

Merged
piggggggggy merged 6 commits intodevelopfrom
feature-console-query-enhancement
Apr 25, 2025
Merged

fix(opsflow): apply latest console query#5819
piggggggggy merged 6 commits intodevelopfrom
feature-console-query-enhancement

Conversation

@piggggggggy
Copy link
Member

Skip Review (optional)

  • Minor changes that don't affect the functionality (e.g. style, chore, ci, test, docs)
  • Previously reviewed in feature branch, further review is not mandatory
  • Self-merge allowed for solo developers or urgent changes

Description (optional)

Things to Talk About (optional)

piggggggggy and others added 5 commits April 24, 2025 15:40
Signed-off-by: piggggggggy <samuel.park@mz.co.kr>
Signed-off-by: piggggggggy <samuel.park@mz.co.kr>
Signed-off-by: piggggggggy <samuel.park@mz.co.kr>
Signed-off-by: piggggggggy <samuel.park@mz.co.kr>
@piggggggggy piggggggggy added the self_approved/review Pull Request has been reviewed and approved by the author without requiring additional reviewers. label Apr 25, 2025
@vercel
Copy link

vercel bot commented Apr 25, 2025

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

4 Skipped Deployments
Name Status Preview Comments Updated (UTC)
console ⬜️ Ignored (Inspect) Visit Preview Apr 25, 2025 1:56am
dashboard ⬜️ Ignored (Inspect) Visit Preview Apr 25, 2025 1:56am
feature-integration-project-detail ⬜️ Ignored (Inspect) Visit Preview Apr 25, 2025 1:56am
web-storybook ⬜️ Ignored (Inspect) Visit Preview Apr 25, 2025 1:56am

@github-actions
Copy link
Contributor

⚠️ @piggggggggy the signed-off-by was not found in the following 1 commits:

  • 0ffcd05: Revert "feat(console-query): create tanstack query debugger"

✅ Why it is required

The Developer Certificate of Origin (DCO) is a lightweight way for contributors to certify that they wrote or otherwise have the right to submit the code they are contributing to the project. Here is the full text of the DCO.

Contributors sign-off that they adhere to these requirements by adding a Signed-off-by line to commit messages.

This is my commit message

Signed-off-by: Random Developer <randomdeveloper@example.com>

Git even has a -s command line option to append this automatically to your commit message:

$ git commit -s -m 'This is my commit message'

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR applies updates to the console query functionality by removing the deprecated useAPIQueryKey hook and its associated query key parameters from multiple API client composables. Key changes include:

  • Removal of the useAPIQueryKey hook in all relevant API client modules.
  • Refactoring of API action definitions from asynchronous functions to direct assignments (e.g. in cost-analysis API clients).
  • Renaming of the PublicConfig interface for delete operations.

Reviewed Changes

Copilot reviewed 86 out of 86 changed files in this pull request and generated no comments.

Show a summary per file
File Description
apps/web/src/api-clients/identity/role/composables/use-role-api.ts Removed unused role query keys.
apps/web/src/api-clients/identity/role-binding/composables/use-role-binding-api.ts Removed unused role-binding query keys.
apps/web/src/api-clients/identity/provider/composables/use-provider-api.ts Removed unused provider query keys.
apps/web/src/api-clients/identity/project/composables/use-project-api.ts Removed unused project query keys.
apps/web/src/api-clients/identity/project-group/composables/use-project-group-api.ts Removed unused project group query keys.
apps/web/src/api-clients/identity/package/composables/use-package-api.ts Removed unused package query keys.
apps/web/src/api-clients/identity/job/composables/use-job-api.ts Removed unused job query keys.
apps/web/src/api-clients/identity/endpoint/composables/use-endpoint-api.ts Removed unused endpoint query keys.
apps/web/src/api-clients/identity/domain/composables/use-domain-api.ts Removed unused domain query keys.
apps/web/src/api-clients/identity/app/composables/use-app-api.ts Removed unused app query keys.
apps/web/src/api-clients/identity/agent/composables/use-agent-api.ts Removed unused agent query keys.
apps/web/src/api-clients/cost-analysis/data-source/composables/use-data-source-api.ts Refactored API actions to direct assignments.
apps/web/src/api-clients/cost-analysis/cost-report-config/composables/use-cost-report-config-api.ts Refactored API actions to direct assignments.
apps/web/src/api-clients/config/user-config/composables/use-user-config-api.ts Removed unused user config query keys.
apps/web/src/api-clients/config/shared-config/composables/use-shared-config-api.ts Removed unused shared config query keys.
apps/web/src/api-clients/config/public-config/schema/api-verbs/delete.ts Updated interface name to PublicConfigDeleteParameters.
apps/web/src/api-clients/config/public-config/composables/use-public-config-api.ts Removed unused public config query keys and updated imports.
apps/web/src/api-clients/config/domain-config/composables/use-domain-config-api.ts Removed unused domain config query keys.
apps/web/src/api-clients/_common/types/query-key-type.ts Removed obsolete query key type definitions.
apps/web/src/api-clients/_common/composables/use-api-query-key.ts Removed the unused query key composable.

Signed-off-by: samuel.park <samuel.park@megazone.com>
@github-actions
Copy link
Contributor

⚠️ @piggggggggy the signed-off-by was not found in the following 1 commits:

  • 0ffcd05: Revert "feat(console-query): create tanstack query debugger"

✅ Why it is required

The Developer Certificate of Origin (DCO) is a lightweight way for contributors to certify that they wrote or otherwise have the right to submit the code they are contributing to the project. Here is the full text of the DCO.

Contributors sign-off that they adhere to these requirements by adding a Signed-off-by line to commit messages.

This is my commit message

Signed-off-by: Random Developer <randomdeveloper@example.com>

Git even has a -s command line option to append this automatically to your commit message:

$ git commit -s -m 'This is my commit message'

@piggggggggy piggggggggy merged commit 1f6beba into develop Apr 25, 2025
11 of 12 checks passed
@piggggggggy piggggggggy deleted the feature-console-query-enhancement branch April 25, 2025 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fail/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.

2 participants