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

The Package#authors field should attempt to drop any invalid author entries rather than strictly fail validation. #9533

Closed
aaclayton opened this issue May 30, 2023 · 0 comments
Assignees
Labels
data-models Issues related to data models and schema changes packages Issues related to add-on packages, systems, modules, and worlds

Comments

@aaclayton
Copy link
Contributor

User feedback:

Also, apparently anything converted from roll20 with R20Converter is suffering from this same issue due to the authors field being incorrect, and that's likely tens of thousands of worlds that will simply just not launch in v11 without causing huge hardship to users who will have absolutely no idea why it's happening or even an inkling on how to fix it. If a value in authors doesn't have the name field, then drop it, instead of blocking world launch. If you see an invalid compendium pack, then consider that it isn't set, instead of blocking world launch, and show a warning to the user, or add a suffix automatically to make it unique

@aaclayton aaclayton added this to the V11 - Stable Maintenance milestone May 30, 2023
@aaclayton aaclayton added packages Issues related to add-on packages, systems, modules, and worlds data-models Issues related to data models and schema changes labels May 30, 2023
@aaclayton aaclayton self-assigned this May 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data-models Issues related to data models and schema changes packages Issues related to add-on packages, systems, modules, and worlds
Projects
Status: Done
Development

No branches or pull requests

1 participant