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

add pubsub redis config docs #1334

Merged
merged 6 commits into from
Apr 9, 2021

Conversation

Taction
Copy link
Member

@Taction Taction commented Mar 24, 2021

Thank you for helping make the Dapr documentation better!

Please follow this checklist before submitting:

  • Read the contribution guide
  • Commands include options for Linux, MacOS, and Windows within codetabs
  • New file and folder names are globally unique
  • Page references use shortcodes instead of markdown or URL links
  • Images use HTML style and have alternative text
  • Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:

Description

add some config that can be set from metadata for redis client.

Issue reference

#1339

@mukundansundar
Copy link
Contributor

@Taction Please create a docs issue and link this PR to that ...

@pkedy pkedy self-assigned this Mar 25, 2021
Copy link
Member

@pkedy pkedy left a comment

Choose a reason for hiding this comment

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

The new settings are all capture. Great job!

Made some wording tweaks / suggestions but @orizohar is probably better at that :)

Also, I think dapr/components-contrib#763 needs to be merged before this can be.

@orizohar orizohar changed the base branch from v1.1 to v1.2 March 26, 2021 20:28
@orizohar
Copy link

@Taction - since the relevant feature is going in v1.2 (not the upcoming v1.1) I rebased the PR so it is merged to the v1.2 docs branch

@github-actions
Copy link

github-actions bot commented Apr 1, 2021

Stale PR, paging all reviewers

@github-actions github-actions bot added the stale label Apr 1, 2021
@AaronCrawfis AaronCrawfis added the requested-change PR needs an update or change as requested by the maintainer(s) label Apr 6, 2021
@AaronCrawfis AaronCrawfis merged commit 0808724 into dapr:v1.2 Apr 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
requested-change PR needs an update or change as requested by the maintainer(s) stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants