Skip to content

fix: name editions sensibly when the library gives a placeholder name - #41

Open
scooper4711 wants to merge 1 commit into
mainfrom
fix/placeholder-edition-names
Open

scooper4711 wants to merge 1 commit into
mainfrom
fix/placeholder-edition-names

Conversation

@scooper4711

Copy link
Copy Markdown
Owner

Some entries carry a placeholder name such as "Test" or "Accidental duplicate package - disregard" while their file is the real book. Such an edition now takes the name of a properly named edition of the same kind, with its update date added, or else a name built from the product name, so its label and file name make sense.

Changes:

  • Add PlaceholderNames to detect placeholder edition names.
  • EntitlementGrouper replaces a placeholder name with the name of a properly named edition of the same kind (plus its update date), falling back to a name built from the product name.
  • Update the library manager design spec and add unit tests for the naming rules.

Closes #40

Some entries carry a placeholder name such as "Test" or "Accidental
duplicate package - disregard" while their file is the real book. Such
an edition now takes the name of a properly named edition of the same
kind, with its update date added, or else a name built from the product
name, so its label and file name make sense.

Closes #40
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.

Name editions sensibly when Paizo's listing gives a placeholder name

1 participant