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

Added yWorks related tools #18

Merged
merged 2 commits into from
May 21, 2018
Merged

Added yWorks related tools #18

merged 2 commits into from
May 21, 2018

Conversation

yGuy
Copy link
Contributor

@yGuy yGuy commented Mar 16, 2018

This adds links to the visualization library with the neo4j JavaScript visualization demo, and the free neo4j explorer web-app.

This adds links to the visualization library with the neo4j JavaScript visualization demo, and the free neo4j explorer web-app.
@yGuy
Copy link
Contributor Author

yGuy commented Mar 16, 2018

OK - the check fails, because I have the same link in two different context twice in the document, if I see this correctly. Feel free to remove the first link or tell me if this is how you want to resolve this issue. Thx!

the build failed because of a duplicate link (used in different contexts)
@yGuy
Copy link
Contributor Author

yGuy commented Mar 16, 2018

I removed the duplicate links. Still it complains because of redirects (but not because of my links, only). I don't think it is a good idea to directly use the long urls. The redirects are there to be used as permanent links. Let me know if you want me to resolve this.

@disolovyov disolovyov requested a review from FylmTM May 21, 2018 10:39
@FylmTM FylmTM merged commit 2b0d15b into neueda:master May 21, 2018
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 this pull request may close these issues.

2 participants