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

feat: update API call to use inventory V2 #5196

Conversation

sulmoJ
Copy link
Member

@sulmoJ sulmoJ commented Dec 10, 2024

To Reviewers

  • Self-reviewed (coding conventions, bug-free, functionality verified, tests checked, documentation updated)
  • Minor change, review optional (style, chore, ci, straightforward changes, etc.)
  • Previously reviewed in feature branch, no further review needed
  • Need review or discussion

Description (optional)

Things to Talk About (optional)

- Replace direct store access with `useUserStore` for language.
- Update components to use computed property for language.

Signed-off-by: sulmo <[email protected]>
- Changed all job-related API calls from `inventory` to `inventoryV2`.
- Updated methods in various components and stores for consistency.

Signed-off-by: sulmo <[email protected]>
- Corrected the API endpoint to use the updated version.
- Ensured compatibility with the new inventoryV2 client.

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

vercel bot commented Dec 10, 2024

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

4 Skipped Deployments
Name Status Preview Comments Updated (UTC)
bye-bye-vuex ⬜️ Ignored (Inspect) Visit Preview Dec 10, 2024 8:58am
console ⬜️ Ignored (Inspect) Visit Preview Dec 10, 2024 8:58am
dashboard ⬜️ Ignored (Inspect) Visit Preview Dec 10, 2024 8:58am
hotfix2 ⬜️ Ignored (Inspect) Visit Preview Dec 10, 2024 8:58am

@sulmoJ sulmoJ merged commit 8e31e8d into cloudforet-io:feature-project-alert-manager Dec 10, 2024
8 of 9 checks passed
@sulmoJ sulmoJ deleted the feature-project-alert-manager branch December 10, 2024 09:04
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.

1 participant