Skip to content

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

Merged
rdeltour merged 1 commit into
masterfrom
fix/1230/multiple-roles
Jun 29, 2021
Merged

rdeltour merged 1 commit into
masterfrom
fix/1230/multiple-roles

Conversation

@rdeltour

@rdeltour rdeltour commented Jun 29, 2021 •

Copy link
Copy Markdown
Member

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 requested a review from mattgarrish June 29, 2021 12:01
@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

⌂ 换网址