fastlane-old / deliver

Upload screenshots, metadata and your app to the App Store using a single command
https://fastlane.tools
2.24k stars 162 forks source link

'default' release_notes in Deliverfile gives error on tunes_client.rb:207:in `handle_itc_response': [!] There are one or more errors on the page. #583

Closed owurman closed 8 years ago

owurman commented 8 years ago

If I have the following in my Deliverfile:

release_notes({ 'default '=> "Testing" })

Then I get the following error when running deliver --skip_screenshots --skip_binary_upload :

/Library/Ruby/Gems/2.0.0/gems/spaceship-0.19.4/lib/spaceship/tunes/tunes_client.rb:207:in handle_itc_response': [!] There are one or more errors on the page. (Spaceship::TunesClient::ITunesConnectError) from /Library/Ruby/Gems/2.0.0/gems/spaceship-0.19.4/lib/spaceship/tunes/tunes_client.rb:347:inupdate_app_version!' from /Library/Ruby/Gems/2.0.0/gems/spaceship-0.19.4/lib/spaceship/tunes/app_version.rb:269:in save!' from /Library/Ruby/Gems/2.0.0/gems/deliver-1.10.2/lib/deliver/upload_metadata.rb:130:inverify_available_languages!' from /Library/Ruby/Gems/2.0.0/gems/deliver-1.10.2/lib/deliver/upload_metadata.rb:22:in upload' from /Library/Ruby/Gems/2.0.0/gems/deliver-1.10.2/lib/deliver/runner.rb:58:inupload_metadata' from /Library/Ruby/Gems/2.0.0/gems/deliver-1.10.2/lib/deliver/runner.rb:21:in run' from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.59.0/lib/fastlane/actions/deliver.rb:18:inrun' from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.59.0/lib/fastlane/runner.rb:143:in block (2 levels) in execute_action' from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.59.0/lib/fastlane/actions/actions_helper.rb:35:inexecute_action' from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.59.0/lib/fastlane/runner.rb:129:in block in execute_action' from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.59.0/lib/fastlane/runner.rb:128:inchdir' from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.59.0/lib/fastlane/runner.rb:128:in execute_action' from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.59.0/lib/fastlane/fast_file.rb:149:inmethod_missing' from (eval):83:in block (2 levels) in parsing_binding' from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.59.0/lib/fastlane/lane.rb:36:incall' from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.59.0/lib/fastlane/lane.rb:36:in call' from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.59.0/lib/fastlane/runner.rb:52:inblock in execute' from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.59.0/lib/fastlane/runner.rb:46:in chdir' from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.59.0/lib/fastlane/runner.rb:46:inexecute' from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.59.0/lib/fastlane/lane_manager.rb:46:in cruise_lane' from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.59.0/lib/fastlane/command_line_handler.rb:30:inhandle' from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.59.0/bin/fastlane:38:in block (2 levels) in run' from /Library/Ruby/Gems/2.0.0/gems/commander-4.3.5/lib/commander/command.rb:178:incall' from /Library/Ruby/Gems/2.0.0/gems/commander-4.3.5/lib/commander/command.rb:178:in call' from /Library/Ruby/Gems/2.0.0/gems/commander-4.3.5/lib/commander/command.rb:153:inrun' from /Library/Ruby/Gems/2.0.0/gems/commander-4.3.5/lib/commander/runner.rb:428:in run_active_command' from /Library/Ruby/Gems/2.0.0/gems/fastlane_core-0.36.4/lib/fastlane_core/ui/fastlane_runner.rb:23:inrun!' from /Library/Ruby/Gems/2.0.0/gems/commander-4.3.5/lib/commander/delegates.rb:15:in run!' from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.59.0/bin/fastlane:150:inrun' from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.59.0/bin/fastlane:156:in <top (required)>' from /usr/bin/fastlane:23:inload' from /usr/bin/fastlane:23:in `

'

Removing the release_notes fixes the issue. I have fully updated fastlane, deliver and spaceship.

In the verification html, it shows:

en-US Changelog Screenshots Skipped Screenshots

Options to skip_screenshots activated. deliver will not modify your app screenshots.

owurman commented 8 years ago

The same error reproduces when passing the release_notes directly as a hash in my Fastfile using the key 'en-us', but 'en-US' worked. So it might be an error with a bad key, but isn't 'default' a valid key for the hash per the documentation?

https://github.com/fastlane/deliver

vpolouchkine commented 8 years ago

Looks like you have an extra space in 'default '=> "Testing".

owurman commented 8 years ago

So it does. I'll try again later today when I'm submitting another update, using 'default' properly. However, it would be great if keys could be checked for validity and a meaningful error returned. I'm happy to look at doing this.

vpolouchkine commented 8 years ago

Go for it. :+1:

owurman commented 8 years ago

So I tried using "default" again. No crash (yay) but it didn't change anything for en-US and eventually error'd with

/Library/Ruby/Gems/2.0.0/gems/spaceship-0.21.0/lib/spaceship/tunes/tunes_client.rb:219:in `handle_itc_response': [!] You must detail what's new in this version in English. (Spaceship::TunesClient::ITunesConnectError)

I guess I don't understand what default is supposed to do, if not fill in for any language that isn't explicitly defined?

vpolouchkine commented 8 years ago

Perhaps @ConfusedVorlon can chime in?

ConfusedVorlon commented 8 years ago

@owurman - which build are you using? The default functionality was broken in the last release, though it has been fixed in head. I'm getting on a plane shortly, so I'll be unavailable for about 48 hrs, but I can check properly when I get to the other end.

What you're doing looks correct - the idea is that 'default' is just treated like a language key in the hash.If you're willing to debug, then have a look at `upload_metadata.rb/assign_defaults``

also worth checking with a file-system approach. Create a /default folder in your metadata folder (right next to your other languages), and put release_notes.txt (check that the name is correct) with 'test' as the content

owurman commented 8 years ago

I was using 1.63.1 which probably has that broken functionality. I'll wait for the next release and try again.

fastlane-bot commented 8 years ago

This issue was migrated to https://github.com/fastlane/fastlane/issues/1523. Please post all further comments there.

fastlane is now a mono repo, you can read more about the change in our blog post. All tools are now available in the fastlane main repo :rocket: