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.

Sign up for GitHub

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

Jump to bottom

Created topic: Nebula Graph CRUD #163

Merged
merged 11 commits into from
Oct 27, 2020
Merged

Created topic: Nebula Graph CRUD #163

merged 11 commits into from
Oct 27, 2020

Conversation

randomJoe211
Copy link
Contributor

No description provided.

@randomJoe211 randomJoe211 added doc affected Solution: improvements or additions to documentation wip Solution: work in progress labels Oct 13, 2020
whitewum
whitewum previously approved these changes Oct 13, 2020
View reviewed changes
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
whitewum
whitewum suggested changes Oct 16, 2020
View reviewed changes
Copy link
Contributor

@whitewum whitewum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the index is a great pitfall, and we have much much experience in the index section.
put and write exactly the same place and same contont as 1.0 manual.

docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
@randomJoe211
Copy link
Contributor Author

. You can ==> . You can

Sorry, I can't see what this comment refers to... Could you please add comments in the lines? Thank you!

oldLady344
oldLady344 reviewed Oct 22, 2020
View reviewed changes
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
@randomJoe211 randomJoe211 removed the wip Solution: work in progress label Oct 23, 2020
oldLady344
oldLady344 reviewed Oct 23, 2020
View reviewed changes
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated
Got 1 rows (time spent 2006/2406 us)
```

* Use `LOOKUP`: Find the information of the vertex with the tag `player` and have the name `"Tony Parker"`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"have the name ..." should have been part of an adjective clause.

docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
whitewum
whitewum suggested changes Oct 27, 2020
View reviewed changes
Copy link
Contributor

@whitewum whitewum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

more to come

docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
@randomJoe211 randomJoe211 removed the doc affected Solution: improvements or additions to documentation label Oct 27, 2020
whitewum
whitewum reviewed Oct 27, 2020
View reviewed changes
Copy link
Contributor

@whitewum whitewum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

still some minors

docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
whitewum
whitewum previously approved these changes Oct 27, 2020
View reviewed changes
Copy link
Contributor

@whitewum whitewum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please fix other's comments

docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
docs_2.0/manual_EN/3.quick-start/4.nebua-graph-crud.md Outdated Show resolved Hide resolved
@whitewum whitewum merged commit c4faa78 into vesoft-inc:master Oct 27, 2020
oldLady344 pushed a commit to oldLady344/nebula-docs that referenced this pull request Nov 16, 2020
@hetao92
docs: update version (vesoft-inc#163)
098ede7 
* docs: update  version

* mod: changelog update

* mod: code review
whitewum added a commit that referenced this pull request Nov 16, 2020
@oldLady344 @dutor
@nianiaJR @wilsonyou @whitewum @amber-moe @NicolaCage @hetao92
Move studio docs (en) (#203)
2227fb6 
* Initial commit

* Merge request for docker (#1)

* feat: first version init

* mod: docs

* fix: docs (#3)

* feat: update docs && web image (#4)

* nebula graph studio user guide (#5)

* user-guide

* details about host ip

* feat: add nginx import file download (#6)

* docs: update (#7)

* feat: release web image to 0.0.7 (#8)

* mod: change relative image always use latest (#9)

* docs: update (#10)

* docs: fix (#11)

* Update README.md (#12)

* mod: changelog (#13)

* en version and cn minor updates (#15)

* Update README.md (#14)

* [doc] links updated (#16)

* en version and cn minor updates

* links updated

* minor update

* [doc]add readme (#17)

* add readme

* address comments

* address comments

* fix typo (#18)

* fix changelog links (#20)

* [doc] minor updates (#19)

* minor updates

* notes added

* minor updates

* [doc] figures added  (#24)

* figures added

* minor updates

* updated

* nGQL

* docs: update changelog (#31)

* docs: update changelog (#34)

* mod: upload to server (#42)

* mod: upload to server

* mod: http client max size

* fix: file download

* docs: fix

* mod: docker image name

Co-authored-by: NicolaCage <37568394+NicolaCage@users.noreply.github.com>

* mod: CHANGELOG (#64)

* docs: update changelog (#66)

* docs: release 1.0.4-beta (#68)

* update-doc (#96)

* update-changelog (#97)

* update-changelog

* update

* docs: add 1.0.5-beta changelog (#98)

* docs: add 1.0.5-beta changelog

* docs: fix typo

* docs: fix typo (#99)

* Release V1.0.6-beta (#102)

* docs: add 1.0.5-beta changelog

* docs: fix typo

* docs: release v1.0.6-beta

* update-doc (#105)

* add-example (#106)

* add-video-link (#110)

* docs: update release (#111)

* chore: release 1.0.8-beta (#116)

* show-edge-doc (#119)

* chore: release 1.0.9-beta (#132)

* update USE (#136)

* docs: release 1.0.10-beta (#142)

* docs: release 1.0.10-beta

* mod: doc review

* docs: release 1.1.0-beta (#143)

* docs: release 1.1.0-beta

* docs: update

* docs: changelog update (#144)

* docs: changelog update

* docs: update

* New user guide (#148)

* added new files for user guide

* deleted doc

* updated on whitewum's comments

* corrected links

* updated what-is-studio

* updated docs

* updated

* corrected links

* updated plan-schema

* updated

* removed uuid

* fixed links and corrected description (#150)

* fixed image links (#151)

* fixed image links

* fixed format

* added schema management (#152)

* fix conflict

* modified st-ug-crud-index.md

* updated

* fixed

* updated toc.md

* fixed format errors

* fixed punctuation errors

* docs: changelog update (#155)

* fixed typo (#154)

* Update readme (#157)

* updated readme.md and re-organized folder

* updated

* added schema management (english) (#159)

* added manage schema (english)

* updated toc.md

* removed company name from docs (#160)

* Update limitations (#161)

* updated limitations

* updated limitations

* updated limitations

* added a link to studio video (#162)

* added a link to studio video

* updated

* added video link for the English README.md

* docs: update  version (#163)

* docs: update  version

* mod: changelog update

* mod: code review

* mod: max size of upload files update (#164)

* added open-in-explorer & updated toc (#165)

* added open-in-explorer & updated toc

* updated

* updated

* Add version info && update limitations (#166)

* added version info into operation docs and fixed errors

* updated

* added version info into operation docs and fixed errors

* updated

* updated

* added view subgraph (#169)

* add English quick start && polish some docs of chinese version (#167)

* added manage schema (english)

* added english quick start && polished the chinese version

* updated toc

* updated

* fixed comments

* Add version info && update limitations (#166)

* added version info into operation docs and fixed errors

* updated

* added version info into operation docs and fixed errors

* updated

* updated

* fixed comments

* added view subgraph (#169)

* added manage schema (english)

* added english quick start && polished the chinese version

* updated toc

* updated

* fixed comments

* fixed comments

* open in explore (english) (#168)

* added Open-in-Explore (en)

* updated

* rename

* added Open-in-Explore (en)

* updated

* rename

* edited toc

* updated version

* fixed typo

* fixed comments

* updated UI description

* fixed comments

Co-authored-by: dutor <440396+dutor@users.noreply.github.com>
Co-authored-by: Jerry <8124783+nianiaJR@users.noreply.github.com>
Co-authored-by: wilsonyou <40747875+wilsonyou@users.noreply.github.com>
Co-authored-by: min.wu <50101159+whitewum@users.noreply.github.com>
Co-authored-by: Amber Zhang <42762957+Amber1990Zhang@users.noreply.github.com>
Co-authored-by: NicolaCage <37568394+NicolaCage@users.noreply.github.com>
Co-authored-by: hetao <hetao.daily@gmail.com>
Co-authored-by: He Nut <18328704+hetao92@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@oldLady344 oldLady344 oldLady344 left review comments

@amber-moe amber-moe amber-moe approved these changes

@whitewum whitewum whitewum approved these changes

@CPWstatic CPWstatic Awaiting requested review from CPWstatic

@jude-zhu jude-zhu Awaiting requested review from jude-zhu

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@randomJoe211 @amber-moe @whitewum @oldLady344