Skip to content

[App Service] az staticwebapp create: Allow creating Static Web Apps not connected to a github repo#22042

Merged
zhoxing-ms merged 1 commit intoAzure:devfrom
StrawnSC:empty_swa
Apr 13, 2022
Merged

[App Service] az staticwebapp create: Allow creating Static Web Apps not connected to a github repo#22042
zhoxing-ms merged 1 commit intoAzure:devfrom
StrawnSC:empty_swa

Conversation

@StrawnSC
Copy link
Copy Markdown
Contributor

Description

Fixes #21058

Testing Guide

az staticwebapp create -g {} -n {}

History Notes


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

@ghost ghost added the Auto-Assign Auto assign by bot label Apr 12, 2022
@ghost ghost requested review from wangzelin007 and yonzhan April 12, 2022 23:33
@ghost ghost assigned zhoxing-ms Apr 12, 2022
@ghost ghost added this to the Apr 2022 (2022-04-26) milestone Apr 12, 2022
@ghost ghost added the App Services az appservice label Apr 12, 2022
@yonzhan
Copy link
Copy Markdown
Collaborator

yonzhan commented Apr 12, 2022

App Service

@panchagnula
Copy link
Copy Markdown
Contributor

@anthonychu & @simonaco FYI


def create_staticsites(cmd, resource_group_name, name, location, # pylint: disable=too-many-locals,
source, branch, token=None,
def create_staticsites(cmd, resource_group_name, name, location="centralus", # pylint: disable=too-many-locals,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did we want to make location optional & Default to CentralUs?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did we want to make location optional & Default to CentralUs?

Hi StrawnSC,
Can you confirm with this ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, this is what @simonaco put on the ADO work item

Copy link
Copy Markdown
Contributor

@panchagnula panchagnula left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM . Given there is no need for GitHub integration we can probably create a test case for empty SWA

@wangzelin007 wangzelin007 changed the title [App Service] az staticwebapp create: Allow creating Static Web Apps not connected to a github repo [App Service] az staticwebapp create: Allow creating Static Web Apps not connected to a github repo Apr 13, 2022
@zhoxing-ms zhoxing-ms merged commit 7978dd4 into Azure:dev Apr 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

App Services az appservice Auto-Assign Auto assign by bot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

StaticWebApp:az staticwebapp create - skip GitHub workflow generation

5 participants