-
Notifications
You must be signed in to change notification settings - Fork 16
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
Remove create dtxsid by api #995
Conversation
…lace-vcr-with-recording
…romatron/hawc into replace-vcr-with-recording
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. I think you need to fix your vscode so it stops adding trailing commas; its hard to review b/c there are a bunch of unintended formatting changes.
I think we need a test where the dtxsid is invalid, like dtxsid0000000000 or something like that?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removes the ability to create DSSTox model through Epi Exposure API, as validation does not properly pull data.
create
method was added to DSSTox viewset which properly populates ccte content correctly for dtxisd. Modified and created tests.