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

Resolve remaining failures in #1823 #1838

Merged

Conversation

candleindark
Copy link
Member

@candleindark candleindark commented Jan 27, 2024

This PR resolves the remaining failures in tests for backend in #1823 once dandi/dandi-schema#218 is merged to the master of the dandi-schema repo, a new version of dandi-schema is published, and set to be used by dandi-archive.

 The validator for the `assetsSummary` field in the
 `PublishedDandiset` model in dandischema 0.9.0
 generates an error with a slightly different message
 The validator for the `digest` field in the
 `PublishedAsset` model in dandischema 0.9.0
 generates an error with a slightly different message
 than before
@candleindark
Copy link
Member Author

@mvandenburgh I don't have experience with Vue. Would you take a look at this PR to see if you can resolve the failures in the frontend tests? I suspect the failures in those tests are result of the changes in the JSON schema. You can locate the documentation of those changes at the top posts of dandi/dandi-schema#203 and dandi/dandi-schema#218.

@yarikoptic
Copy link
Member

Great -- thank you @candleindark ! indeed only e2e tests remain affected , so let's wait for @mvandenburgh or other @dandi/archive-maintainers web-tech aware folks to chime in before merging into the original PR #1823 for an overall look/review.

@mvandenburgh mvandenburgh self-assigned this Jan 31, 2024
@yarikoptic
Copy link
Member

To minimize number of open PRs, I will progress #1823 to the state in this PR, which will probably make github to close this one.

@yarikoptic yarikoptic merged commit e0f7849 into dandi:rf-new-dandischema Feb 1, 2024
8 of 9 checks passed
@yarikoptic
Copy link
Member

@mvandenburgh feel welcome to self-assign #1823 ;-)

@candleindark candleindark deleted the rf-new-dandischema branch February 1, 2024 23:36
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

Successfully merging this pull request may close these issues.

3 participants