-
Notifications
You must be signed in to change notification settings - Fork 205
Update reference to Python web app quickstart #537
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
Conversation
@rayluo please review the updates to docs quickstarts |
Thanks for the catch. The old link currently shows "... This probably isn't the page you were expecting. While we work on a fix ...". Are we actually working on a fix to restore the old link, or are we permanently moving to the new link? In other words, what will be the canonical link going forward? |
We're moving to the new link that holds all quickstarts for a particular app type. Soon we'll remove all references to the old quickstarts and delete them. However, we'll also apply redirects to ensure users do not land on already deleted pages. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the catch and clarifying that the new link is the new canonical link.
<!-- Don't know how to include images into Sphinx, so we host it from github repo instead --> | ||
<img src="https://raw.githubusercontent.com/AzureAD/microsoft-authentication-library-for-python/dev/docs/scenarios-with-users.svg" | ||
usemap="#public-map"><!-- Derived from http://www.image-map.net/ but we had to manually add unique map id --> | ||
<map name="public-map"> | ||
<area target="_blank" coords="110,150,59,94" shape="rect" | ||
alt="Web app" title="Web app" href="https://docs.microsoft.com/en-us/azure/active-directory/develop/quickstart-v2-python-webapp"> | ||
alt="Web app" title="Web app" href="https://learn.microsoft.com/azure/active-directory/develop/web-app-quickstart?pivots=devlang-python> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It turns out, this line and the next line were both missing a trailing quotation mark.
Now I fixed it in my latest commit.
CC: @Dickson-Mwendia
Fixes a couple of broken links, update from https://learn.microsoft.com/en-us/azure/active-directory/develop/quickstart-v2-python-webapp to https://learn.microsoft.com/en-us/azure/active-directory/develop/web-app-quickstart?pivots=devlang-python