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

Endpoint unreachable #24

Open
CedricProfessionnel opened this issue Feb 1, 2021 · 6 comments
Open

Endpoint unreachable #24

CedricProfessionnel opened this issue Feb 1, 2021 · 6 comments
Labels
bug Something isn't working

Comments

@CedricProfessionnel
Copy link
Contributor

Steps to reproduce :

  • do a getSample from the datasetManangers

Why its happen :
catch the wrong endpoint on the server

image

@CedricProfessionnel CedricProfessionnel added the bug Something isn't working label Feb 1, 2021
@seveibar
Copy link
Contributor

seveibar commented Feb 2, 2021

I'm not sure this issue has enough detail, getSample does work sometimes- otherwise the collaborative sessions wouldn't work at all.

What commands were done to create the sample?

@CedricProfessionnel
Copy link
Contributor Author

CedricProfessionnel commented Feb 2, 2021

This is my test no change in setSample getSample

new CollaborationServer() //no param
.createNewSession //Necessary to return true on isReady
.isReady() // wait until return true
.setDataset() //send a new dataset as a param
var testSample = { _id: "test" }
.setSample("sdjz2g1qa", testSample) //"sdjz2g1qa" is created in the previous dataset
.getSample("sdjz2g1qa") || .getSample("test") // I tried both

.

@CedricProfessionnel
Copy link
Contributor Author

CedricProfessionnel commented Feb 2, 2021

Remark :
The error message above is from endpoints/get-session.js
and i want to catch endpoints/get-sample.js

@CedricProfessionnel
Copy link
Contributor Author

CedricProfessionnel commented Feb 2, 2021

@seveibar
Copy link
Contributor

seveibar commented Feb 2, 2021

oh interesting! So it seems we have a path typo :/

@CedricProfessionnel
Copy link
Contributor Author

I don't know it could be since both result in error but if i check router.js it don't seem to have an "s"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants