Skip to content

Conversation

@jiansong-msft
Copy link
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
Member

qwordy commented Apr 27, 2021

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

@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