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.

Sign up for GitHub

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

Jump to bottom

Test audiannotate #426

Merged

Dananji merged 2 commits into main from test-audiannotate

Feb 22, 2024

Collaborator

Dananji commented Feb 21, 2024

Related issue: #346

Changes in the parser:

Fail gracefully for empty canvases when represented with a body but no URIs in it
Fix condition for reading items list under AnnotationPage property, handle cases when there are no items list

These changes are related to reading AudiAnnotate manifests in Ramp.

Dananji Withana and others added 2 commits

February 20, 2024 14:50


          Fixes to parser to read AV Annotate Manifests

78ea7db


          Add empty Canvas handling and tests for parser changes

05c1ec8

masaball approved these changes

View reviewed changes

Contributor

masaball left a comment

Looks good!

Dananji merged commit 18df945 into main

2 checks passed

Dananji deleted the test-audiannotate branch

February 22, 2024 19:48

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet