Mimeo / VSTSExtension-ActivePullRequests

An extension for Azure DevOps that provides a better view of active Pull Requests in a project.
Other
32 stars 11 forks source link

After today's update (v1.1.1) it stopped working, error in js console #53

Open admg opened 5 years ago

admg commented 5 years ago

Not showing pull requests, there's an error in js console in browser. TFS is on premise.

image

bundle.js:53 Uncaught (in promise) TFS.WebApi.Exception: 404: Not Found at Object.<anonymous> (http://***:8080/tfs/_apis/public/gallery/publisher/mimeo-vs-marketplace/extension/mimeo-active-pull-requests/1.1.1/assetbyname/dist/bundle.js:53:51961) at http://***:8080/tfs/_apis/public/gallery/publisher/mimeo-vs-marketplace/extension/mimeo-active-pull-requests/1.1.1/assetbyname/dist/bundle.js:53:50486 at Object.next (http://***:8080/tfs/_apis/public/gallery/publisher/mimeo-vs-marketplace/extension/mimeo-active-pull-requests/1.1.1/assetbyname/dist/bundle.js:53:50591) at a (http://***:8080/tfs/_apis/public/gallery/publisher/mimeo-vs-marketplace/extension/mimeo-active-pull-requests/1.1.1/assetbyname/dist/bundle.js:53:49337)

johngraumann commented 5 years ago

We are seeing the same exception. We are running Azure DevOps Server 2019.

joelbourbon commented 5 years ago

This is the part in code that probably breaks on premise: image

It also would not work for teams still using the old URLs (organization.visualstudio.com).

Not sure with what it can be replaced....

joelbourbon commented 5 years ago

Here is a great article on how URLs should be managed in extensions: https://docs.microsoft.com/en-us/azure/devops/extend/develop/work-with-urls?view=azure-devops&tabs=nodejsgeneric

joelbourbon commented 5 years ago

Oh sorry guys I was wrong. What I found is probably causing other issues.

The bug you found seems to be related to the BuildRestClient. The way it is used is fairly straightforward and I don't see how this could be fixed for On-Premise. Seems like an issue in the underlying library used here --> azure-devops-extension-api

persalmi commented 5 years ago

Is anyone working on a resolution for this problem? The hard reference to dev.azure.com seems easy to fix but is it the whole truth for getting the extension to work on-premise again?

wadechandler commented 4 years ago

I'm seeing a definite issue with the cross domain problem mentioned above by @joelbourbon and we have a different URL we access from yet still the SaaS version (as we've been on ADO a long time). It seems the URL the user is already using in the browser should be used versus the hard coded version as mentioned above.