-
Notifications
You must be signed in to change notification settings - Fork 2
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
Some manifests cause crash #109
Comments
I don't lose more time on it, I only remove this breaking manifest from default catalog. |
Yes @geourjoa Lets keep this open and see if we figure out what's going on. The Manifest is announced as using Presentation API v3 (the last) so it should not be considered as an "old" manifest. Could the japanese characters inside it be the reason ? |
Ok, i have a lead I downgrade manifesto.js 4.20.20 to 4.20.17 and it seem to work https://www.npmjs.com/package/manifesto.js?activeTab=versions |
I need also to force the addition of multiple package. npm install react-sizeme
npm i @babel/plugin-proposal-private-property-in-object
npm i @babel/plugin-proposal-object-rest-spread
npm i I use currently node 18 on this project |
This manifest can break MAE only in beeing in the manifest catalog.
"https://dzkimgs.l.u-tokyo.ac.jp/videos/iiif_in_japan_2017/manifest.json"
I think it is the old manifest format
To reproduce :
App crash on manifest.js L142
The text was updated successfully, but these errors were encountered: