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

List documents on the screen #3

Merged
merged 2 commits into from
Apr 17, 2022
Merged

List documents on the screen #3

merged 2 commits into from
Apr 17, 2022

Conversation

hackerwins
Copy link
Member

@hackerwins hackerwins commented Apr 11, 2022

What this PR does / why we need it?

I implemented ListDocuments API one of the Admin APIs to Yorkie in yorkie-team/yorkie#309.
In this PR, we call ListDocuments to display a list of documents on the screen.

  • List documents on the screen.
  • Remove counter and add documents feature
  • Leave counter feature for reference

Any background context you want to provide?

You need backend servers like Yorkie and Envoy to run Yorkie House. You can simply run them using docker-compose in your project root.

$ docker-compose -f docker/docker-compose.yml up --build -d

And you can create test documents using yorkie-js-sdk. To create them, start co-editing example in your yorkie-js-sdk.

$ npm start
$ open HTTP://0.0.0.0:9000/

What are the relevant tickets?

Address yorkie-team/yorkie#273

Checklist

  • Added relevant tests or not required
  • Didn't break anything

- Leave counter feature for reference
@hackerwins hackerwins changed the title Remove counter and add documents feature List documents on the screen Apr 13, 2022
@hackerwins hackerwins marked this pull request as draft April 13, 2022 04:46
@hackerwins hackerwins marked this pull request as ready for review April 14, 2022 02:20
@hackerwins hackerwins merged commit e006e3b into main Apr 17, 2022
@hackerwins hackerwins deleted the grpc-web branch April 17, 2022 10:03
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.

1 participant