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

yaml validator for presentations.yaml #142

Open
castrojo opened this issue Aug 17, 2023 · 4 comments
Open

yaml validator for presentations.yaml #142

castrojo opened this issue Aug 17, 2023 · 4 comments
Assignees

Comments

@castrojo
Copy link
Member

We need a validator for presentations.yaml before we open it up to submissions.

@cjyabraham
Copy link
Contributor

As part of this make the slides parameter required since I key off of it during the import.

@castrojo
Copy link
Member Author

castrojo commented Oct 2, 2023

What do we do in cases where there's no slides, video only? Do we just make having slides a prerequisite?

@cjyabraham
Copy link
Contributor

We had previously discussed making the slides field required. I am now keying off that field during the import process. If we don't make it required, then I'll need to key off some other field that is always required and guaranteed to be unique for each presentation. Alternatively, I could come up with some kind of composite field, made up of video + slides fields, to use as a unique key. Any thoughts?

@jeefy
Copy link
Member

jeefy commented Oct 8, 2023

@cjyabraham I'd do slides+date as a composite field. We don't currently require videos (but if we could I can add that too and we can add that to said composite field)

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

No branches or pull requests

3 participants