Skip to content

[AppService] BREAKING CHANGE: Add support for az staticwebapp show; az staticwebapp create; az staticwebapp update#17870

Merged
qwordy merged 14 commits intoAzure:devfrom
jiansong-msft:azswa
Apr 27, 2021
Merged

[AppService] BREAKING CHANGE: Add support for az staticwebapp show; az staticwebapp create; az staticwebapp update#17870
qwordy merged 14 commits intoAzure:devfrom
jiansong-msft:azswa

Conversation

@jiansong-msft
Copy link
Copy Markdown
Contributor

@jiansong-msft jiansong-msft commented Apr 26, 2021

Description

  1. az staticwebapp browse -> az staticwebapp show
  2. add option of sku for az staticwebapp create --sku
  3. add command of az staticwebapp update

Testing Guide

Tested commands - working as expected:
1. az staticwebapp show -n my-first-static-web-app
2. az staticwebapp create -n my-second-static-web-app -g rg -s https://github.com/jiansong-msft/my-first-static-web-app -l CentralUS -b main --token github-token --sku Free
3. az staticwebapp update -n my-second-static-web-app --tags "key=value"
4. az staticwebapp update -n static-app-not-exists throws
5. with invalid sku options passed in throws


This checklist is used to make sure that common guidelines for a pull request are followed.

@jiansong-msft jiansong-msft changed the title [AppService] Add support for Static WebApps: show, create, update [AppService] az staticwebapp show; az staticwebapp create; az staticwenapp update Apr 26, 2021
@jiansong-msft jiansong-msft changed the title [AppService] az staticwebapp show; az staticwebapp create; az staticwenapp update [AppService] Add support for az staticwebapp show; az staticwebapp create; az staticwenapp update Apr 26, 2021
@jiansong-msft jiansong-msft marked this pull request as draft April 26, 2021 16:14
@jiansong-msft jiansong-msft marked this pull request as ready for review April 26, 2021 16:15
@calvinsID calvinsID added this to the S186 milestone Apr 26, 2021
@calvinsID calvinsID requested a review from miwebst April 26, 2021 16:59
@qwordy
Copy link
Copy Markdown
Member

qwordy commented Apr 27, 2021

Minor issue, CLIError is an old style. ValidationError is preferred.

fix typo

Co-authored-by: Feiyue Yu <iamyfy@163.com>
@qwordy qwordy changed the title [AppService] Add support for az staticwebapp show; az staticwebapp create; az staticwenapp update [AppService] Add support for az staticwebapp show; az staticwebapp create; az staticwebapp update Apr 27, 2021
@jiansong-msft jiansong-msft changed the title [AppService] Add support for az staticwebapp show; az staticwebapp create; az staticwebapp update [AppService] BREAKING CHANGE: Add support for az staticwebapp show; az staticwebapp create; az staticwebapp update Apr 27, 2021
@jiansong-msft jiansong-msft changed the title [AppService] BREAKING CHANGE: Add support for az staticwebapp show; az staticwebapp create; az staticwebapp update [AppService] Add support for az staticwebapp show; az staticwebapp create; az staticwebapp update Apr 27, 2021
@jiansong-msft jiansong-msft changed the title [AppService] Add support for az staticwebapp show; az staticwebapp create; az staticwebapp update [AppService] BREAKING CHANGE: Add support for az staticwebapp show; az staticwebapp create; az staticwebapp update Apr 27, 2021
@qwordy qwordy merged commit 8c04af8 into Azure:dev Apr 27, 2021
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.

5 participants