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

Discussion: future development #319

Closed
3 of 5 tasks
mojoaxel opened this issue Sep 9, 2019 · 12 comments
Closed
3 of 5 tasks

Discussion: future development #319

mojoaxel opened this issue Sep 9, 2019 · 12 comments
Assignees
Labels

Comments

@mojoaxel
Copy link
Member

mojoaxel commented Sep 9, 2019

🎉 I'm happy that ngx-vis is now part of the visjs family!

We should discuss the future of this repository and how we could integrate it more.

Here are my first thoughts as a basis for a discussion:

  1. I would say one of the first steps should be to move the dependency from vis to vis-timeline and vis-network. see switch to the new visjs libraries #314
  2. The typescript types from vis-timeline and vis-network should be used in the future. @types/vis is also deprecated.
  3. I think the next step should be to seperate ngx-vis into it's modules e.g. @visjs/ngx-vis-timeline and @visjs/ngx-network. This could e.g. be done with a monorepo approach!?
  4. The "donate" section of the README should be removed in favor of a opencollective badge.
  5. cicle-ci? auto-release? code-formating?
@Thomaash
Copy link
Member

Thomaash commented Sep 9, 2019

Reexported stuff from Timeline and Network will have accompanying types. I don't see any problem here. The first two bullets should be fairly simple.

@hypery2k
Copy link
Collaborator

hypery2k commented Sep 9, 2019

I've already created a branch which can be used for 1. - 3. https://github.com/visjs/ngx-vis/tree/feature/visjs-integration

Regarding 3. Yes I would go the monorepo approach. How should the npm package be named?
@visjs/ngx or @visjs/angular ?

  1. Done

  2. I currently use my own Jenkins server and travis. Jenkins will produce also snapshots for npm package. If we want to switch I can live with that ;)

@mojoaxel
Copy link
Member Author

mojoaxel commented Sep 9, 2019

Regarding 3. Yes I would go the monorepo approach. How should the npm package be named?
@visjs/ngx or @visjs/angular ?

I think the trick about the monorepo ist that the sources stay together but seperated modules can be releases out of one repository. I would recommend schema: @visjs/ngx-vis-timeline and @visjs/ngx-network ...

@hypery2k
Copy link
Collaborator

For the new types I've started a PR: #321

Seems to require some more changes besides simple switching imports

hypery2k added a commit that referenced this issue Nov 19, 2019
@hypery2k
Copy link
Collaborator

Just published version 3.0.0 and have also demo. Should we publish it also to the project homepage?

hypery2k added a commit that referenced this issue Dec 3, 2019
@mojoaxel
Copy link
Member Author

Should we publish it also to the project homepage?

Because it is not an independent project but "just" another way to integrate the existing libraries I would recommend to add an "Angualar-Button" to timeline and network on visjs.org.
Also It could be useful to add a sentence about this angular component to the READMEs of those libraries.

@hypery2k
Copy link
Collaborator

completly overlooked your last comment (@mojoaxel): So should we add it on the README ?

@mojoaxel
Copy link
Member Author

mojoaxel commented Oct 5, 2020

I think we should add this to https://github.com/visjs/awesome-visjs

@ericleib
Copy link

Regarding 3): How about just exporting just 2 modules instead of 1 (VisModule):

  • VisNetworkModule exports VisNetworkDirective
  • VisTimelineModule exports VisTimelineDirective

This simple change would allow to use the network without carrying the timeline into the bundle.

@hypery2k
Copy link
Collaborator

sounds great. Feel free to send a PR for that

@dsl400
Copy link

dsl400 commented Mar 9, 2023

In angular 14 When importing VisTimelineDirective in a standalone component it errors out
Component imports must be standalone components, directives, pipes, or must be NgModules.

@hypery2k
Copy link
Collaborator

a new preview is available via npm i ngx-vis@next

For now, I will not have much time for further work on this, so closing this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants