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

update vis.js #192

Closed
mojoaxel opened this issue Sep 15, 2020 · 1 comment · Fixed by #209
Closed

update vis.js #192

mojoaxel opened this issue Sep 15, 2020 · 1 comment · Fixed by #209

Comments

@mojoaxel
Copy link

mojoaxel commented Sep 15, 2020

vis is depecated!
You should switch to the new an up-to-date vis-timeline!

just-at-uber added a commit that referenced this issue Oct 22, 2020
* fixed timeline dependency to use latest version. Current version is depricated. see issue #192

* fixing failing integration test
@mojoaxel
Copy link
Author

🎉

just-at-uber added a commit that referenced this issue Oct 23, 2020
* removing encodeURIComponent to avoid double decoding URL

* locking all versions so they dont auto fetch latest minor versions

* revert package file fix

* fixed timeline dependency to use latest version. Current version is depricated. see issue #192

* fixing failing integration test

* only escape forward slash

* adding console logs to help debugging

* moving logic into helper and fixing at router level.

* adding unit tests

* removing escaped runId as this is generated from server and is gauranteed to be a UUID
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

Successfully merging a pull request may close this issue.

2 participants