-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Add Email Crud Operations using Az Dev Tools #7169
Add Email Crud Operations using Az Dev Tools #7169
Conversation
|
rule | cmd_name | rule_message | suggest_message |
---|---|---|---|
communication email create | cmd communication email create added |
||
communication email delete | cmd communication email delete added |
||
communication email domain | sub group communication email domain added |
||
communication email list | cmd communication email list added |
||
communication email show | cmd communication email show added |
||
communication email update | cmd communication email update added |
||
communication email wait | cmd communication email wait added |
Hi @ddouglas-msft, |
Thank you for your contribution! We will review the pull request and get back to you soon. |
Hello @AllyW. Any updates? |
src/communication/azext_communication/aaz/latest/communication/email/__cmd_group.py
Show resolved
Hide resolved
@ddouglas-msft please also make a pr in aaz repo (https://github.com/Azure/aaz) with the code changes in dev env |
@@ -4,7 +4,7 @@ | |||
# -------------------------------------------------------------------------------------------- | |||
|
|||
|
|||
VERSION = '1.8.0b1' | |||
VERSION = '1.8.1b1' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Based on our CLI Extension Version Transition Table, the version should be 1.8.0b2
.
Please also update azext.isPreview
to true
in azext_metadata.json as well if you want to release as preview version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This has been updated per your other PR feedback
The updates are in this PR. |
Please fix CI issues |
Thanks. I have fixed the issue with the indent. |
Co-authored-by: Yu Chen <16348853+jsntcy@users.noreply.github.com>
[Release] Update index.json for extension [ communication ] : https://dev.azure.com/azclitools/internal/_build/results?buildId=132640&view=results |
Adds operations for the Azure Communication Services Email Service
This checklist is used to make sure that common guidelines for a pull request are followed.
Related command
General Guidelines
azdev style <YOUR_EXT>
locally? (pip install azdev
required)python scripts/ci/test_index.py -q
locally? (pip install wheel==0.30.0
required)For new extensions:
About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update
src/index.json
automatically.You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify
src/index.json
.