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

schemas: Review all to ensure *schema* defines things as correctly as possible. #200

Open
Athanasius opened this issue Oct 5, 2022 · 0 comments

Comments

@Athanasius
Copy link
Contributor

The general policy is that, as much as is feasible, a schema file alone should be enough to inform a Sender as to how to construct a compliant message.

outfitting/2, at least, has some shortcomings. It's explicit about excluding some things, but doesn't actually stop int_planetapproachsuite from being Relayed.

So, review all the schemas, tightening things up as per existing documentation.

In terms of branch/PR this should be done with a PR per schema. There's bound to be some changes that will need very careful consideration versus current Sender behaviour (especially taking into account where both CAPI and Journal data sources can be used). We don't want to hold up good changes because we're still hashing out the details of more troublesome ones.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant