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

Error loading manifest TypeError: Cannot read properties of undefined (reading 'locations') #57

Open
xibsked opened this issue Sep 30, 2024 · 6 comments

Comments

@xibsked
Copy link

xibsked commented Sep 30, 2024

I have cloned the current develop branch and successfully compiled the test app. The app opens in the browser, and I can see parts of the UI. However, my publications are not loading, and I see the following error in the console.

image

Below screen comes on when it loads

image

Below is the manifest

"https://streamer.edrlab.org/pub/L3N0cmVhbWVyL21pc2MvZXB1YnMvd2FzdGVsYW5kLW90Zi1vYmYuZXB1Yg%3D%3D/manifest.json

@HadrienGardeur
Copy link
Contributor

The current version of the toolkit seems to expect to find a positions list which is missing from this example.

@xibsked
Copy link
Author

xibsked commented Sep 30, 2024

I am using r2-streamer-js, and I'm getting a manifest.json similar to the one I shared. The links node does not have any reference to the position list, as required by this ts-toolkit. How can I obtain the position list from the publication server?

@danielweck
Copy link
Member

hello, positionslist is not implemented by the r2-streamer-js

@HadrienGardeur
Copy link
Contributor

I recommend using the go-toolkit for your back-end for the time being, but we'll take this requirement into account for future revisions of the ts-toolkit and playground.

@xibsked
Copy link
Author

xibsked commented Sep 30, 2024

Thank you for the recommendation. However, using go-toolkit is challenging for us right now, as we have additional customizations built on top of r2-streamer-js. Could you please advise if there’s a way to retain r2-streamer-js in the backend as it currently is?

@HadrienGardeur
Copy link
Contributor

Sure, you could add the Positions List to r2-streamer-js, that's doable too.

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