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

Remove the 'printings' field #65

Closed
Sembiance opened this Issue Aug 3, 2015 · 1 comment

Comments

Projects
None yet
1 participant
@Sembiance
Collaborator

Sembiance commented Aug 3, 2015

With the introduction of the 'printingCodes' field, the 'printings' field is just duplicate information, in a much longer format.

This makes the '-x' files much larger than they need to be.

Making this change would likely break some apps/websites that use this field, until they update their code to the new printingCodes field, which is why I haven't done this already.

@Sembiance Sembiance added the discuss label Aug 17, 2015

@Sembiance Sembiance closed this in 11538b9 Aug 18, 2015

@Sembiance

This comment has been minimized.

Show comment
Hide comment
@Sembiance

Sembiance Aug 18, 2015

Collaborator

The 'printingsCode' field has been removed.

Now the 'printings' field is a list of set codes instead of set names.

Collaborator

Sembiance commented Aug 18, 2015

The 'printingsCode' field has been removed.

Now the 'printings' field is a list of set codes instead of set names.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment