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

Postgen modal azure links don't redirect correctly #1062

Closed
jsondoo opened this issue Aug 9, 2019 · 17 comments
Closed

Postgen modal azure links don't redirect correctly #1062

jsondoo opened this issue Aug 9, 2019 · 17 comments
Assignees
Labels
bug Something is broken external-issue Wizard Wizard issue

Comments

@jsondoo
Copy link

jsondoo commented Aug 9, 2019

The links we have for the 3 services are:

  1. Functions: https://portal.azure.com/#blade/WebsitesExtension/FunctionsIFrameBladeMain
  2. Cosmos DB: https://portal.azure.com/#blade/HubsExtension/BrowseResourceBlade/resourceType/Microsoft.DocumentDb%2FdatabaseAccounts
  3. App Service: https://ms.portal.azure.com/#blade/HubsExtension/BrowseResourceBlade/resourceType/Microsoft.Web%2Fsites

CosmosDB and App Service link takes you to the "All Resources" page rather than the "Cosmos DB" or "App Service" resource group pages when clicked on through the VSCode extension.

VSCode Issue: microsoft/vscode#45515
VSCode currently decodes the %2F into a slash, so it takes you to a different path.

@SahilTara
Copy link
Collaborator

By the way you might want to change ms.portal.azure.com/ to portal.azure.com

@jsondoo jsondoo added the bug Something is broken label Aug 9, 2019
@SahilTara
Copy link
Collaborator

Hmm odd, not sure if i understand the issue.
image
is this not the right page? (Clicked on client side)

@jsondoo
Copy link
Author

jsondoo commented Aug 12, 2019

@SahilTara that is the right page, it seems to work when you click on the link on the running it on the browser, but for some reason it takes you somewhere else when you click the link through the vscode extension

@SahilTara
Copy link
Collaborator

Hmm that's really strange

@jsondoo

This comment has been minimized.

@jsondoo jsondoo added Blocked Tasks that are blocked and removed Blocked Tasks that are blocked labels Aug 19, 2019
@SahilTara
Copy link
Collaborator

SahilTara commented Aug 19, 2019

Think you mean to add microsoft/vscode#45515 (if you scroll down in the link you posted they track decoding on the linked issue).

@jsondoo
Copy link
Author

jsondoo commented Aug 20, 2019

Think you mean to add microsoft/vscode#45515 (if you scroll down in the link you posted they track decoding on the linked issue).

Thanks, updated

@sibille sibille added external-issue and removed Blocked Tasks that are blocked labels Nov 11, 2019
@sibille
Copy link
Collaborator

sibille commented Nov 11, 2019

VSCode tracking issue is microsoft/vscode#83645

@sibille sibille added the Wizard Wizard issue label Nov 11, 2019
@jrieken
Copy link
Member

jrieken commented Nov 15, 2019

@sibille How do you open these links? Via vscode.env.openExternal or as links that the editor detects?

@sibille
Copy link
Collaborator

sibille commented Nov 26, 2019

@jrieken I'd say a link that the editor detects: Link is shown in a react app running in the webview. Link rendering code in question is :

@jrieken
Copy link
Member

jrieken commented Nov 26, 2019

Can you try with latest insiders? That should have a fix for these issues

@sibille
Copy link
Collaborator

sibille commented Nov 27, 2019

I tested with Insiders 1.41 and this seems still broken.
Source html is
<a href="https://portal.azure.com/#blade/HubsExtension/BrowseResourceBlade/resourceType/Microsoft.Web%2Fsites" class="styles_link__3M7Qr">View</a>

but then shows as

image

@jrieken
Copy link
Member

jrieken commented Nov 27, 2019

The link that get's shown there must not be the link that gets opened, e.g the encoding in the dialog is likely different but I assume that opening is also not working, right? Is that html from an embedded webview?

@sibille
Copy link
Collaborator

sibille commented Nov 27, 2019

I just double checked, the link is the same in the browser, so its not opening correct.

Is that html from an embedded webview?

Yes, Web Template Studio is shown in the embedded webview

@jrieken
Copy link
Member

jrieken commented Nov 29, 2019

check - this should be better with next (Mondays) insiders build, microsoft/vscode#83645 (comment)

@sibille
Copy link
Collaborator

sibille commented Dec 2, 2019

Working now on todays insider build.
Version: 1.41.0-insider (user setup)
Commit: 7cf4cca47aa025a590fc939af54932042302be63

The link is still showing up wrong in the popup, but leads to the correct place
image
image
Thanks @jrieken!

@sibille
Copy link
Collaborator

sibille commented Apr 29, 2020

Verified on dev-nightly:
Templates version: 0.2.20120.1
Wizard version: 0.2.2012001

@sibille sibille closed this as completed Apr 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is broken external-issue Wizard Wizard issue
Projects
None yet
Development

No branches or pull requests

5 participants