Skip to content

Standardize messages for CRUD commands#1381

Merged
Brian Strauch (brianstrauch) merged 2 commits intomainfrom
resource-msgs
Jul 31, 2022
Merged

Standardize messages for CRUD commands#1381
Brian Strauch (brianstrauch) merged 2 commits intomainfrom
resource-msgs

Conversation

@brianstrauch

Checklist

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

What

Add a few more resources to the resource package and use them to standardize CRUD messages: Always end messages with a period and put resources in quotes.

Test & Review

Updated unit and integration tests.

@brianstrauch Brian Strauch (brianstrauch) requested a review from a team as a code owner July 30, 2022 03:43
Copy link
Contributor

@DABH David Hyde (DABH) left a comment

Choose a reason for hiding this comment

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

Seems harmless as far as I can tell... thank you!

@brianstrauch Brian Strauch (brianstrauch) deleted the resource-msgs branch July 31, 2022 20:58
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