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

feat: allow multiple roles for creator, contributor, and publisher #1258

Merged
merged 1 commit into from
Jun 29, 2021

Conversation

rdeltour
Copy link
Member

@rdeltour rdeltour commented Jun 29, 2021

This PR proactively implements the relaxed EPUB 3.3 rules for the role property, to fix #1230.

In EPUB 3.2, the 'role' property was only allowed to refine 'contributor'
or 'creator' properties, and only once per property. A new check was
introduced in EPUBCheck v4.2.5, and some users reported that it cause
issues with existing EPUB catalogs.

EPUB 3.3 will relax this by allowing authors to attach several roles to
creators, contributors, and publishers.

This commit proactively implements the relaxed EPUB 3.3 rules.

Fixes #1230
@rdeltour rdeltour added the status: needs review Needs to be reviewed by a team member before further processing label Jun 29, 2021
@rdeltour rdeltour added this to the v4.2.6 milestone Jun 29, 2021
@rdeltour rdeltour self-assigned this Jun 29, 2021
@rdeltour rdeltour merged commit 6c68c61 into master Jun 29, 2021
@rdeltour rdeltour deleted the fix/1230/multiple-roles branch June 29, 2021 13:39
shlomif pushed a commit to shlomif/epubcheck that referenced this pull request Jul 31, 2021
…3c#1258)

In EPUB 3.2, the 'role' property was only allowed to refine 'contributor'
or 'creator' properties, and only once per property. A new check was
introduced in EPUBCheck v4.2.5, and some users reported that it cause
issues with existing EPUB catalogs.

EPUB 3.3 will relax this by allowing authors to attach several roles to
creators, contributors, and publishers.

This commit proactively implements the relaxed EPUB 3.3 rules.

Fixes w3c#1230
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: needs review Needs to be reviewed by a team member before further processing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow multiple roles for creator, contributor and publisher
2 participants