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

feat(kafka): add AVRO type in message key binding #81

Merged
merged 2 commits into from
Sep 15, 2021

Conversation

M3lkior
Copy link
Contributor

@M3lkior M3lkior commented Sep 13, 2021

Refer to asyncapi/avro-schema-parser#67

Here is the related update due to the support of avro file in kafka key bindings

@M3lkior M3lkior force-pushed the feat/add-avro-to-kafka-bindings-key branch from 418d7d4 to ed839e2 Compare September 13, 2021 15:55
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Welcome to AsyncAPI. Thanks a lot for creating your first pull request. Please check out our contributors guide and the instructions about a basic recommended setup useful for opening a pull request.

Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.

kafka/README.md Outdated Show resolved Hide resolved
@M3lkior M3lkior force-pushed the feat/add-avro-to-kafka-bindings-key branch from 106af09 to d4218b4 Compare September 14, 2021 10:16
Co-authored-by: Fran Méndez <[email protected]>
@M3lkior M3lkior force-pushed the feat/add-avro-to-kafka-bindings-key branch from d4218b4 to 8b4e853 Compare September 14, 2021 10:17
Copy link
Member

@fmvilas fmvilas left a comment

Choose a reason for hiding this comment

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

Looks good to me 👍

Copy link
Member

@magicmatatjahu magicmatatjahu left a comment

Choose a reason for hiding this comment

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

LGTM, but we must remember about changing it in the future 😄

@M3lkior
Copy link
Contributor Author

M3lkior commented Sep 14, 2021

LGTM

LGTM, but we must remember about changing it in the future 😄

yes, but it allows to fix the regression from the nunjunks template to the react refactoring :)

@fmvilas fmvilas merged commit 68f9b2f into asyncapi:master Sep 15, 2021
@magicmatatjahu
Copy link
Member

@M3lkior Sorry 😅 What do you mean by:

yes, but it allows to fix the regression from the nunjunks template to the react refactoring :)

?

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.

3 participants