-
Notifications
You must be signed in to change notification settings - Fork 91
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 outdated hugegraph download link #211
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot, I do remember there are several outdated download links like this.
Could you help to search for them in full-text and fix them together? In addition, the doc of the English directory (current is cn
) also needs to be corrected
Also could add the basic instruction in hubble doc (visualize graph)
sure, i'm glad to do this! now i deploy hugegraph and hubble by docker and try to run the loader demo i'll add some basic instructions in hubble doc in next PR. One question, When hubble creates a graph, we need to specify a
|
Also i noticed that #447 modify the default UI, should we wait for the docker environment to take effect before modifying the screenshot. |
Hi, @imbajin, i have fixed outdate links, But some locations are not sure, please help me to have a look, thanks! |
find here: |
seem works: server ok
create graph insert data
|
Current loader doc is also too long to read (users may get lost in the page), it's not friendly and the doc should separate into multi subpages. (show the most important info for user first) [TODO]
GraphID & GraphName is really confused for new users to set the proper value, also need enhance the description & the doc [TODO]
No need, the docker images could update anytime (currently it build once per week, we hope there is a way to auto build it when PR merged or enable a dockerhub hook)
fine |
And we should also add a groovy script in hubble's doc so that users could add a graph with Just like the groovy example in server separate in another pr |
@@ -40,7 +40,7 @@ brew install wget | |||
# 4. 下载 hugegraph-svn 目录 (版本号注意填写此次验证版本, 这里以1.0.0为例) | |||
svn co https://dist.apache.org/repos/dist/dev/incubator/hugegraph/1.0.0/ | |||
# (注) 如果出现 svn 下载某个文件速度很慢的情况, 可以考虑 wget 单个文件下载, 如下 (或考虑使用 VPN / 代理) | |||
wget https://dist.apache.org/repos/dist/dev/incubator/hugegraph/1.0.0/apache-hugegraph-toolchain-incubating-1.0.0.tar.gz | |||
wget https://downloads.apache.org/incubator/hugegraph/1.0.0/apache-hugegraph-toolchain-incubating-1.0.0.tar.gz |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also need revert here (svn file could also be download by http mode)
* fix hugegraph-loader download link --------- Co-authored-by: imbajin <[email protected]> 50c327d
as title
the original url got 404 error, replace it with url from here