[Alexandru Cristu (acristu)]

quickfix for #529

[Sébastien Levert (sebastienlevert)]

Thanks for your contribution Alexandru Cristu (@acristu)! We will be reviewing and merge when ready! This will be available in our next version of the SDK!

[Alexandru Cristu (acristu)]

no pb, sry this was in a hurry, don't have time to run all tests and make a proper PR

actually on my end I just modified the js file dirrectly in node_modules, I need to do a quick PoC ... so for me this workaround is sufficient for now...

[Nikitha Chettiar (nikithauc)]

Alexandru Cristu (@acristu) Thank you so much for this contribution!

Unfortunately at present, the library expects an encoded URI to be passed as a parameter. We have customers passing in an encoded URI and I would recommend this as a workaround now.

I labelled this PR as a breaking change considering cases of encoding already encoded URLs which would break some customers.

[Alexandru Cristu (acristu)]

Hi Nikitha Chettiar (@nikithauc) , thanks for the reply, so this means the 3min Graph intro node.js app should be fixed to send startDateTime/endDateTime in URI encoded format ? I was following the intro when I hit this issue ... the downloaded app did not work for me ...

[Nikitha Chettiar (nikithauc)]

Hi Nikitha Chettiar (@nikithauc) , thanks for the reply, so this means the 3min Graph intro node.js app should be fixed to send startDateTime/endDateTime in URI encoded format ? I was following the intro when I hit this issue ... the downloaded app did not work for me ...

Thanks for bringing this to attention! Let me verify this and make the necessary updates:)

[Nikitha Chettiar (nikithauc)]

#823