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
[Spaceship] Ability to use existing IAP screenshots #10903
[Spaceship] Ability to use existing IAP screenshots #10903
Conversation
When modifying the version parameter, the existing contentHosting data was previously lost, causing an error when saving.
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed, please reply here (e.g.
|
@googlebot I signed it! |
CLAs look good, thanks! |
Looks like some sort of environment error on CircleCI caused the build to fail, will need an admin to try and rebuild it.
|
Looks like circleci is having issues today |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
📸
Congratulations! 🎉 This was released as part of fastlane 2.68.1 🚀 |
Checklist
bundle exec rspec
from the root directory to see all new and existing tests passbundle exec rubocop -a
to ensure the code style is validMotivation and Context
Fixes issues #9826, #9630 and possibly #10743.
The current code does not allow editing an iOS IAPs (in-app purchases) without uploading a new screenshot as well. This change uses the existing uploaded screenshots by default unless the user specifies that they wish to upload a new one.
This can be tested by retrieving an existing IAP and immediately re-saving it. The current code will fail with an
OPERATION_FAILED
response due to the missing screenshots and a bug with thecontentHosting
variable missing. The new code fixes this bug.Description
I've added a public accessor
review_screenshot
to allow users to retrieve the previously uploaded screenshot data. The new code now re-uses the retrieved values for the review screenshot by default, as well as reuses the existingcontentHosting
values which were previously missing when editing and causing saving errors.