-
Notifications
You must be signed in to change notification settings - Fork 33
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
AGILE-3713: Design endpoint listing by filter ID message schemas in 20/EFMP #133
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe it could be better to use italic text style or some other text style to highlight the type of response message in this sentence:
"Repository MUST publish endpoint list by filter response message using the following NATS subject".
And it could be good to use the same style to highlight request/response types in other places.
0020/README.md
Outdated
kaa.v1.replica.{client-service-replica-id}.efmp.ep-list-by-filter-request | ||
``` | ||
|
||
Endpoint filters request message payload MUST be an [Avro-encoded](https://avro.apache.org/) object with the following schema ([0020-endpoint-list-by-filter-request.avsc](./0020-endpoint-list-by-filter-request.avsc)): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change request message type to "Endpoint list by filter"
0020/README.md
Outdated
*Endpoint list by filter response* message MUST be sent by repository in response to an [endpoint list by filter request](#endpoint-list-by-filter-request). | ||
Repository MUST publish endpoint list by filter response message to the subject provided in the NATS `replyTo` field of the request. | ||
|
||
Endpoint filters response message payload MUST be an Avro-encoded object with the following schema ([0020-endpoint-list-by-filter-response.avsc](./0020-endpoint-list-by-filter-response.avsc)): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change response message type to "Endpoint list by filter"
# Conflicts: # 0020/0020-endpoint-list-by-filter-response.avsc # 0020/README.md
No description provided.