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

Update service environment to be a ref #80

Merged
merged 1 commit into from
Oct 19, 2022

Conversation

afrittoli
Copy link
Contributor

Changes

The service environment is a ref to an environment in the schemas and it should be a ref, i.e. a combination of id and source because the id is only unique when combined to the source.

As in other cases, if the source of the environment is the same as the source of the main subject (the service), it may be omitted.

The spec only includes today the environmentId as string - update it to be an object (reference to environment).

Signed-off-by: Andrea Frittoli andrea.frittoli@gmail.com

Submitter Checklist

As the author of this PR, please check off the items in this checklist:

The service environment is a ref to an environment in the schemas
and it should be a ref, i.e. a combination of id and source because
the id is only unique when combined to the source.

As in other cases, if the source of the environment is the same as
the source of the main subject (the service), it may be omitted.

The spec only includes today the environmentId as string - update
it to be an object (reference to environment).

Signed-off-by: Andrea Frittoli <andrea.frittoli@gmail.com>
@afrittoli afrittoli merged commit 8201649 into cdevents:main Oct 19, 2022
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.

2 participants