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

Device specific warnings in S3 HTML template #5775

Merged
merged 3 commits into from
Aug 17, 2016

Conversation

ChrisChares
Copy link
Contributor

@ChrisChares ChrisChares commented Aug 16, 2016

This PR does two things:

  • Identifies supported device types from the .ipa plist file
  • Adds device specific warnings into the default S3 HTML template

Discussed in issue #5774

@@ -110,7 +110,8 @@ def self.run(config)
build_num = info['CFBundleVersion']
bundle_id = info['CFBundleIdentifier']
bundle_version = info['CFBundleShortVersionString']
title = info['CFBundleName']
title = info['CFBundleName']

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Trailing whitespace detected.

@ohayon
Copy link
Contributor

ohayon commented Aug 17, 2016

Hey @ChrisChares! Thanks for the PR! Would you mind sharing a screenshot of this in action?

@ChrisChares
Copy link
Contributor Author

@ohwutup Certainly!

So we've gone from this, displayed for everything, including iPad only apps
image

To this (iPad only app)

  • Desktop
    image
  • iPhone
    image
  • iPad
    image

And this (iPhone app)

  • Desktop
    image
  • iPad
    image
  • iPhone
    image

Note that I changed the verbiage for iPhone only and Universal apps to state you should install on an "iOS device" instead of just "iPhone", as any app marked iPhone only can still be installed on iPads and iPods.

You can play around with these sites yourself. Both were generated with my local copy of fastlane running this code.

@ohayon
Copy link
Contributor

ohayon commented Aug 17, 2016

Thanks for this @ChrisChares! This is awesome! 🚀

@ohayon ohayon merged commit cd94e67 into fastlane:master Aug 17, 2016
asfalcone pushed a commit that referenced this pull request Aug 18, 2016
[10:47:00]: Changes since release 1.100.0:
* Reduce output when running fastlane -v (#5787)
* Device specific warnings in S3 HTML template (#5775)
* Add a flag dry_run_email flag to lane generated by init beta (#5772)
* typo: sepcific -> specific (#5769)
* Choose non interactive (#5738)
* Add Circle.md (#5750)
* Fixed Scan `destination` parameter (#5744)
* [fastlane] latest_testflight_build_number use itc_team_id (#5725)
* Do not fail if no version exists on iTunes Connect (#5590)
* Add groups support to init beta (#5723)
* Skip GitHub issues by default for user_error! (#5622)
* [fastlane] Update fastlane_core dependency (#5721)
* [fastlane] Fix spelling mistake (#5724)
* Add ignore option to commit_version_bump action. (#5541)
* Fix #5571, check the absence of 'manifest' in export_options_plist (#5586)
* [fastlane] Update metadata of git_tag_exists (#5714)
* Disable the rubocop rules for MethodLength and AbcSize (#5715)
* Add option to log output for `import_certificate` per #1314 (#5321)
* Make commander great again (#5612)
* (feat/actions): Add new action 'check if the given git tag exists in the repo or not' (#5555)
* Fix documentation about zip action (#5625)
@asfalcone asfalcone mentioned this pull request Aug 18, 2016
asfalcone pushed a commit that referenced this pull request Aug 18, 2016
[10:47:00]: Changes since release 1.100.0:
* Reduce output when running fastlane -v (#5787)
* Device specific warnings in S3 HTML template (#5775)
* Add a flag dry_run_email flag to lane generated by init beta (#5772)
* typo: sepcific -> specific (#5769)
* Choose non interactive (#5738)
* Add Circle.md (#5750)
* Fixed Scan `destination` parameter (#5744)
* [fastlane] latest_testflight_build_number use itc_team_id (#5725)
* Do not fail if no version exists on iTunes Connect (#5590)
* Add groups support to init beta (#5723)
* Skip GitHub issues by default for user_error! (#5622)
* [fastlane] Update fastlane_core dependency (#5721)
* [fastlane] Fix spelling mistake (#5724)
* Add ignore option to commit_version_bump action. (#5541)
* Fix #5571, check the absence of 'manifest' in export_options_plist (#5586)
* [fastlane] Update metadata of git_tag_exists (#5714)
* Disable the rubocop rules for MethodLength and AbcSize (#5715)
* Add option to log output for `import_certificate` per #1314 (#5321)
* Make commander great again (#5612)
* (feat/actions): Add new action 'check if the given git tag exists in the repo or not' (#5555)
* Fix documentation about zip action (#5625)
@fastlane fastlane locked and limited conversation to collaborators Feb 4, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants