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

Update to use latest_receipt_info instead of receipt #45

Closed
wants to merge 2 commits into from
Closed

Update to use latest_receipt_info instead of receipt #45

wants to merge 2 commits into from

Conversation

tj-alomoves
Copy link

This requires many fewer changes than trying to adapt receipt to work
with the new version of apple receipts; just drop some fields

This requires many fewer changes than trying to adapt receipt to work
with the new version of apple receipts; just drop some fields
For some reason this has always been broken so we've been going
through attributes in our web-alomoves repo... seems to make more
sense to fix this here
@coveralls
Copy link

coveralls commented Mar 21, 2020

Coverage Status

Coverage decreased (-4.3%) to 94.148% when pulling 2f9735b on alomoves:update-for-new-receipts into aa190df on jnbt:master.

@tj-alomoves
Copy link
Author

tj-alomoves commented Mar 21, 2020

I did not mean to make this PR here - this was meant to go to our fork. You're welcome to try to use this or a similar strategy though

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.

None yet

2 participants