Skip to content
This repository has been archived by the owner on Dec 13, 2022. It is now read-only.

combine-*-releases: Warn if no 'records' or 'releases' array #88

Open
jpmckinney opened this issue Apr 15, 2020 · 0 comments
Open

combine-*-releases: Warn if no 'records' or 'releases' array #88

jpmckinney opened this issue Apr 15, 2020 · 0 comments

Comments

@jpmckinney
Copy link
Member

jpmckinney commented Apr 15, 2020

combine_release_packages and combine_record_packages now warn if those required fields are missing: https://ocdskit.readthedocs.io/en/latest/changelog.html#changed

Toucan should catch the warnings (see OCDS Kit docs for the warning class) and present a message to the user.

This is based on seeing a few errors in Sentry.

@jpmckinney jpmckinney added existing command workflow Relating to the user's workflow labels Apr 15, 2020
@jpmckinney jpmckinney added error handling and removed existing command workflow Relating to the user's workflow labels Sep 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant