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

RAML document #4

Closed
jonathan-r-thorpe opened this issue Jan 9, 2024 · 4 comments
Closed

RAML document #4

jonathan-r-thorpe opened this issue Jan 9, 2024 · 4 comments
Labels
enhancement New feature or request triage Apply to all new issues

Comments

@jonathan-r-thorpe
Copy link
Contributor

A RAML document is needed to be consistent with other interface specifications.

@jonathan-r-thorpe jonathan-r-thorpe added the enhancement New feature or request label Jan 9, 2024
@github-actions github-actions bot added the triage Apply to all new issues label Jan 9, 2024
@cristian-recoseanu
Copy link
Contributor

I am not sure it is possible to have a RAML document for this spec since URL paths are implementation specific generated from the device models of each device.

@jonathan-r-thorpe
Copy link
Contributor Author

A good point.

Although the role paths are "URL like" they are not "browsable" like the other IS APIs. i.e. if you browsed to {basePath}/root you get a blob of JSON rather than the members listed as the next level of the URL. As you say, this also makes it hard to define RAML.

Perhaps the {rolePath} should be an atomic parameter (delimited by some character other than /) and therefore is treated as a parameter rather than a URL path, and then it can be expressed in RAML which makes it consistent with the existing interface specifications?

This would also makes the implementation in nmos-cpp easier as it makes it more consistent with existing specs.

@cristian-recoseanu
Copy link
Contributor

We will attempt to do a POC RAML rendition to better determine if this approach would be useful.

@cristian-recoseanu
Copy link
Contributor

Since we've now fully embraced the RAML workflow in our specification is it worth closing this issue @jonathan-r-thorpe?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request triage Apply to all new issues
Projects
None yet
Development

No branches or pull requests

2 participants