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

Update quickstart to include Responder for RaftTypeConfig #1168

Closed
2 of 5 tasks
danthegoodman1 opened this issue Jul 10, 2024 · 2 comments · Fixed by #1181
Closed
2 of 5 tasks

Update quickstart to include Responder for RaftTypeConfig #1168

danthegoodman1 opened this issue Jul 10, 2024 · 2 comments · Fixed by #1181
Assignees
@drmingdrmer

Comments

@danthegoodman1
Copy link
Contributor

What kind of doc do you ask for?

  • Code comments.
  • Architecture design.
  • Algorithm explanation.
  • Guide.
  • Examples.

Describe the information you want
Missing the Responder type for the example in https://docs.rs/openraft/latest/openraft/docs/getting_started/index.html (RaftTypeConfig).
@danthegoodman1 danthegoodman1 changed the title Update quickstart Update quickstart to include responder for RaftTypeCOnfig Jul 10, 2024
@github-actions GitHub Actions
Copy link

👋 Thanks for opening this issue!

Get help or engage by:

  • /help : to print help messages.
  • /assignme : to assign this issue to you.

@danthegoodman1 danthegoodman1 changed the title Update quickstart to include responder for RaftTypeCOnfig Update quickstart to include Responder for RaftTypeConfig Jul 10, 2024
@drmingdrmer drmingdrmer self-assigned this Jul 13, 2024
drmingdrmer added a commit to drmingdrmer/openraft that referenced this issue Jul 20, 2024
@drmingdrmer
Doc: explain default types used by RaftTypeConfig in getting-starte…
037ff4d 
…d.md

- Fix: databendlabs#1168
@drmingdrmer drmingdrmer mentioned this issue Jul 20, 2024
Merged
@drmingdrmer
Copy link
Member

The Responder type is optional and can be left absent. These has been added to getting-started.md in:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@drmingdrmer drmingdrmer

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Doc: explain default types used by RaftTypeConfig in getting-started.md drmingdrmer/openraft
2 participants
@drmingdrmer @danthegoodman1