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

boxCDN failing #249

Closed
addula opened this issue Apr 2, 2018 · 8 comments
Closed

boxCDN failing #249

addula opened this issue Apr 2, 2018 · 8 comments

Comments

@addula
Copy link

addula commented Apr 2, 2018

Please fill out the following template so we can reproduce and fix your issue as quickly as possible!

Note: if your issue includes a potential security vulnerability, please do not file it here. Instead, email the issue to [email protected] for support.

Environment:

  • UI Elements version:
  • Components used:
  • Browser name and version:
  • Operating system and version:

Steps to reproduce the problem:

  1. Try loading preview
  2. Attached image

image

What is the expected behavior? (Screenshots can be helpful here)

What went wrong? (Screenshots can be helpful here)
box cdn call is failing with 404
Link to application or sample code:

If relevant, link to file (or attach file here):

@priyajeet
Copy link
Contributor

You are using the incorrect versions. See the latest ones here https://developer.box.com/docs/box-content-preview

@addula
Copy link
Author

addula commented Apr 2, 2018

I am using Preview.js and Preview.css from LATEST box-ui-elements npm component.

@addula
Copy link
Author

addula commented Apr 2, 2018

@priyajeet

@tonyjin
Copy link
Contributor

tonyjin commented Apr 2, 2018

We do not guarantee every version on npm is deployed to our CDN. Please visit our developer documentation pages (the one Priyajeet linked or https://developer.box.com/docs/box-ui-elements for Elements) to see the latest deployed version.

@addula
Copy link
Author

addula commented Apr 3, 2018

@tonyjin I have dowloaded latest version that is available on NPM and using preview.js, preview.css from dist folder which is generated after npm install box-ui-elements. CDN calls that I posted in the issue are made from preview.js I guess (I think I am sure) which is not in my control. So, now please let me know whether to use box-ui-elements (for preview) or not if there is not guarantee for a stable version.

@addula
Copy link
Author

addula commented Apr 3, 2018

Also, kindly keep the issue in open state unless we wrap up the discussion

@priyajeet
Copy link
Contributor

priyajeet commented Apr 3, 2018

Unless you are building a React app that is trying to import (ES2015) the ui elements in this github repo via NPM, you should not be using instructions mentioned here, nor should you be using NPM. You will need to follow our docs here https://developer.box.com/docs/box-content-preview. Preview and all its required dependencies will come from our CDNs via <script|link>, using path/version as mentioned in the docs link.

If you are building a React app, then this is how preview should be imported https://github.com/box/box-content-preview-demo/blob/master/src/index.js#L8

@tonyjin tonyjin reopened this Apr 3, 2018
@priyajeet
Copy link
Contributor

Closing this for now, discussion can continue on box/box-content-preview#752

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

No branches or pull requests

3 participants