Skip to content

[CIAM-2487] Add warning to KSQL create command#1534

Merged
Lucy Fan (lucy-fan) merged 7 commits intomainfrom
lfan/ciam-2487
Nov 21, 2022
Merged

[CIAM-2487] Add warning to KSQL create command#1534
Lucy Fan (lucy-fan) merged 7 commits intomainfrom
lfan/ciam-2487

Conversation

@lucy-fan
Copy link
Member

Checklist

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

What

References

https://confluentinc.atlassian.net/browse/CIAM-2487

Test & Review

Ran CLI tests and checked that this warning is printed after a successful KSQL create.

@lucy-fan
Copy link
Member Author

Lucy Fan (lucy-fan) commented Nov 18, 2022

Warning text is placeholder for now, everything else is ready for review.

UPDATE: warning text is now the finalized warning text from Anuj and JS.

@lucy-fan Lucy Fan (lucy-fan) marked this pull request as ready for review November 19, 2022 00:26
@lucy-fan Lucy Fan (lucy-fan) requested a review from a team as a code owner November 19, 2022 00:26
@lucy-fan Lucy Fan (lucy-fan) merged commit e4dc0f4 into main Nov 21, 2022
@lucy-fan Lucy Fan (lucy-fan) deleted the lfan/ciam-2487 branch November 21, 2022 20:52
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.

3 participants