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

Only add rasa_environment header to kafka message if it is set #10634

Merged
merged 2 commits into from
Jan 6, 2022
Merged

Only add rasa_environment header to kafka message if it is set #10634

merged 2 commits into from
Jan 6, 2022

Conversation

indam23
Copy link
Contributor

@indam23 indam23 commented Jan 6, 2022

Proposed changes:

  • Only add RASA_ENVIRONMENT header if the env var is set
  • Resolves problem here

Status (please check what you already did):

  • updated the changelog (please check changelog for instructions)
  • reformat files using black (please check Readme for instructions)

@indam23 indam23 requested a review from tayfun January 6, 2022 10:13
@indam23 indam23 requested a review from a team as a code owner January 6, 2022 10:13
@indam23 indam23 mentioned this pull request Jan 6, 2022
Merged
4 tasks
@indam23 indam23 removed the request for review from a team January 6, 2022 10:18
@github-actions
Copy link
Contributor

github-actions bot commented Jan 6, 2022

🚀 A preview of the docs have been deployed at the following URL: https://10634--rasahq-docs-rasa-v2.netlify.app/docs/rasa

@indam23 indam23 merged commit fb0cb86 into main Jan 6, 2022
@indam23 indam23 deleted the fix_kafka_headers branch January 6, 2022 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tayfun tayfun tayfun approved these changes

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

Successfully merging this pull request may close these issues.
2 participants
@indam23 @tayfun