Skip to content
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

How to deploy to App Service #97

Merged
merged 2 commits into from
Mar 28, 2023
Merged

How to deploy to App Service #97

merged 2 commits into from
Mar 28, 2023

Conversation

rayluo
Copy link
Contributor

@rayluo rayluo commented Mar 24, 2023

Hi, @pamelafox , this web app sample itself was already capable to be deployed to App Service. But there are quite some tricky traps during the configuration. I added some know-how based on my recent attempt. Please help review and proofread.

Let me know if you would also want to update some corresponding docs that you recently worked on. I can then add a new tag for you.

README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
app.py Show resolved Hide resolved
Copy link
Contributor

@pamelafox pamelafox left a comment

Choose a reason for hiding this comment

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

Nice to give them ideas for how to deploy! I'm looking into azd-ifying but it looks like there's a blocking issue at the moment (Azure/azure-dev#1714) so this looks like a good approach.

rayluo and others added 2 commits March 28, 2023 11:52
* Pamelas suggestions

* Update requirements.txt

Co-authored-by: Ray Luo <[email protected]>

* Update templates/login.html

Co-authored-by: Ray Luo <[email protected]>

* Ul changes, back button back

* Update templates/login.html

Co-authored-by: Ray Luo <[email protected]>

---------

Co-authored-by: Ray Luo <[email protected]>
@rayluo rayluo merged commit dce9db1 into main Mar 28, 2023
@rayluo rayluo deleted the app-service-howto branch March 28, 2023 19:03
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.

2 participants