Skip to content

Catch v2 error message in List api calls#1440

Merged
MuweiHe merged 2 commits intomainfrom
CLI-1957
Sep 16, 2022
Merged

Catch v2 error message in List api calls#1440
MuweiHe merged 2 commits intomainfrom
CLI-1957

Conversation

@MuweiHe
Copy link
Contributor

@MuweiHe MuweiHe commented Sep 15, 2022

Checklist

  1. [CRUCIAL] Is the change for CP or CCloud functionalities that are already live in prod?
    • yes: ok

What

Catch service-quota commands error message

References

Test & Review

mhe@C02DV0KVMD6T cli % confluent service-quota list cluster               
Error: scope does not exist: 400 Bad Request
mhe@C02DV0KVMD6T cli % confluent service-quota list environment --cluster sdhu
Error: environment scoped quota should not contain filters from other scope: 400 Bad Request

@MuweiHe MuweiHe marked this pull request as ready for review September 15, 2022 22:40
@MuweiHe MuweiHe requested a review from a team as a code owner September 15, 2022 22:40
@MuweiHe MuweiHe changed the title Catch service-quota commands error message Catch v2 error message in List api calls Sep 15, 2022
Copy link

Choose a reason for hiding this comment

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

Thanks! We should do the same for all of the other functions (Get, Update, Delete, etc.)... Although I've tried this and some of them might take a lot of refactoring.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants