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

Stock images url is getting a 404 server error #882

Closed
nviolero opened this issue Apr 28, 2021 · 4 comments
Closed

Stock images url is getting a 404 server error #882

nviolero opened this issue Apr 28, 2021 · 4 comments
Labels
status:fixed-next-drop Issue will be fixed in upcoming release. type:bug
Milestone

Comments

@nviolero
Copy link

Category

[ ] Enhancement

[X] Bug

[ ] Question

Version

Please specify what version of the library you are using: [3.0.0]

Expected / Desired Behavior / Question

The FilePicker control has a tab to get images from Stock Images, but is getting a 404 server error.

Observed Behavior

This is an example of the url that the FilePicker is generating to get the images from Stock images:
https://hubblecontent.osi.office.net/contentsvc/external/m365contentpicker/index.html?p=3&app=1001&aud=prod&channel=devmain&setlang=en-GB&msel=0&env=prod&premium=1&themecolors=%7B%22primaryColor%22%3A%22%23445068%22%2C%22textColor%22%3A%22%23333333%22%2C%22backgroundColor%22%3A%22%23ffffff%22%2C%22neutralLighterColor%22%3A%22%23f3f2f1%22%7D

As you can see, is getting a 404 error. It seems that it's been changed and now the correct one is:
https://hubblecontent.osi.office.net/contentsvc/m365contentpicker/index.html?p=3&app=1001&aud=prod&channel=devmain&setlang=en-GB&msel=0&env=prod&premium=1&themecolors=%7B%22primaryColor%22%3A%22%23445068%22%2C%22textColor%22%3A%22%23333333%22%2C%22backgroundColor%22%3A%22%23ffffff%22%2C%22neutralLighterColor%22%3A%22%23f3f2f1%22%7D

Just take the '/external' from the url.

Steps to Reproduce

Import a FilePicker in your project and try to browse an image using the Stock Images tab.

@ghost
Copy link

ghost commented Apr 28, 2021

Thank you for reporting this issue. We will be triaging your incoming issue as soon as possible.

@ghost ghost added the Needs: Triage 🔍 label Apr 28, 2021
@Employee87
Copy link

Also noticed this too. Thanks for providing the working URL, I have that as a temporary fix now.

AJIXuMuK added a commit that referenced this issue May 1, 2021
@AJIXuMuK AJIXuMuK added status:fixed-next-drop Issue will be fixed in upcoming release. type:bug and removed Needs: Triage 🔍 labels May 1, 2021
@AJIXuMuK
Copy link
Collaborator

AJIXuMuK commented May 1, 2021

Thank you @nviolero for reporting the issue.
It has been fixed and will be included in the next release.

@AJIXuMuK AJIXuMuK added this to the 3.1.0 milestone May 1, 2021
AJIXuMuK added a commit that referenced this issue May 1, 2021
@joelfmrodrigues
Copy link
Collaborator

@nviolero Closing this item as this should now be fixed as part of the last release. Please update to the latest release and open a new issue if you still experience problems.
Many thanks for your contribution

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:fixed-next-drop Issue will be fixed in upcoming release. type:bug
Projects
None yet
Development

No branches or pull requests

4 participants