-
Notifications
You must be signed in to change notification settings - Fork 20
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
changes to service/micro framework #219
Conversation
- removed `api_url` use `metadata` - removed `SCHEMA` verb, schemas can be reported via the endpoint metadata - added `metadata` on the endpoints report (was missing from the ADR)
adr/ADR-32.md
Outdated
- `endpoint` - an optional base endpoint configuration, consisting of valid NATS subject, | ||
handler and optional request/response schema. The `name` of the base endpoint | ||
should be set to `default` by the client libraries. | ||
- `endpoint` - an optional base endpoint configuration, consisting of valid NATS |
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.
we keep this?
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.
Nope - I already took it out.
adr/ADR-32.md
Outdated
handler and optional request/response schema. The `name` of the base endpoint | ||
should be set to `default` by the client libraries. | ||
- `endpoint` - an optional base endpoint configuration, consisting of valid NATS | ||
subject, handler and optional request/response schema. The `name` of the base |
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.
still mentions schema here
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.
the word schema
is about the schema
for the PING response fields - let me remove that
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.
LGTM
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.
LGTM!
api_url
usemetadata
SCHEMA
verb, schemas can be reported via the endpoint metadatametadata
on the endpoints report (was missing from the ADR)