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

fix(data-pipe): export it #78

Merged
merged 2 commits into from
Jan 19, 2020
Merged

fix(data-pipe): export it #78

merged 2 commits into from
Jan 19, 2020

Conversation

Thomaash
Copy link
Member

As per visjs/vis-timeline/issues/254 I noticed some issues. Like the fact that I actually forgot to export this.

This is quite a stupid error but I forgot to export it from the index.
@Thomaash Thomaash requested a review from a team January 18, 2020 22:25
Thomaash added a commit to visjs/vis-timeline that referenced this pull request Jan 18, 2020
Switch from the deprecated type property of Data Set to Data Pipes. This
solves some or maybe all issues from #254.

PS: There is a bug in Vis Data that prevents this from working see
visjs/vis-data/pull/78.
@Thomaash Thomaash merged commit 47429a6 into master Jan 19, 2020
@vis-bot
Copy link
Collaborator

vis-bot commented Jan 19, 2020

🎉 This PR is included in version 6.3.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

Thomaash added a commit to visjs/vis-timeline that referenced this pull request Jan 19, 2020
Switch from the deprecated type property of Data Set to Data Pipes. This
solves some or maybe all issues from #254.

PS: There is a bug in Vis Data that prevents this from working see
visjs/vis-data/pull/78.
@yotamberk yotamberk deleted the fix-data-pipe-export branch January 21, 2020 20:59
Thomaash added a commit to visjs/vis-timeline that referenced this pull request Jan 22, 2020
Switch from the deprecated type property of Data Set to Data Pipes. This
solves some or maybe all issues from #254.

PS: There is a bug in Vis Data that prevents this from working see
visjs/vis-data/pull/78.
yotamberk pushed a commit to visjs/vis-timeline that referenced this pull request Jan 25, 2020
* fix(type-coercion): use Data Pipes

Switch from the deprecated type property of Data Set to Data Pipes. This
solves some or maybe all issues from #254.

PS: There is a bug in Vis Data that prevents this from working see
visjs/vis-data/pull/78.

* docs(type-coercion): add more documentation

* fix(type-coersion): prevent potential NPE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants