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

Support other annotations (W3C Web Annotations) #179

Closed
wants to merge 1 commit into from
Closed

Conversation

ksclarke
Copy link
Owner

@ksclarke ksclarke commented Feb 5, 2023

  • Rework annotations to support adding WC3's annotations (like tagging annotation) -- probably breaking, depending on use
  • Add --add-opens to get rid of jruby/JDK file access warnings

WIP? I'm still looking at this to confirm it's the way I want to go (but I want the CI to run so opening this PR). My question is really just using AnnotationResource instead of the Annotation interface in a few method signatures.

Fixes #178

github-actions[bot]
github-actions bot previously approved these changes Feb 5, 2023
@ksclarke
Copy link
Owner Author

ksclarke commented Feb 6, 2023

Yeah, this is still not right (even though the cookbook tests are passing). Some of those AnnotationResource(s) in the method signatures should be WebAnnotation(s).

github-actions[bot]
github-actions bot previously approved these changes Feb 11, 2023
github-actions[bot]
github-actions bot previously approved these changes Feb 11, 2023
github-actions[bot]
github-actions bot previously approved these changes Feb 11, 2023
github-actions[bot]
github-actions bot previously approved these changes Feb 11, 2023
github-actions[bot]
github-actions bot previously approved these changes Feb 11, 2023
github-actions[bot]
github-actions bot previously approved these changes Feb 12, 2023
github-actions[bot]
github-actions bot previously approved these changes Feb 12, 2023
@ksclarke ksclarke closed this Feb 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant