Skip to content

Move Cancel and Delete to CertificateOperation #8768

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.

Sign up for GitHub

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

Jump to bottom
Merged
heaths merged 4 commits into from
Nov 18, 2019
Merged

Move Cancel and Delete to CertificateOperation #8768

heaths merged 4 commits into from
Nov 18, 2019

Conversation

@heaths
Copy link
Member

@heaths heaths commented Nov 13, 2019

Fixes #8515

@heaths heaths requested review from pakrym and schaabs November 13, 2019 22:42
@heaths
Throw from CertificateOperation.Value for cancel and delete
63bac59 
Based on feedback and design discussions.
@heaths heaths merged commit 79d39d9 into Azure:master Nov 18, 2019
@heaths heaths deleted the issue8515 branch November 18, 2019 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@schaabs schaabs Awaiting requested review from schaabs

@KrzysztofCwalina KrzysztofCwalina Awaiting requested review from KrzysztofCwalina

1 more reviewer

@pakrym pakrym pakrym approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

GetCertificateOperation should be removed

2 participants

@heaths @pakrym