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

[docs][ios] readme: improved “Multi-Deployment Testing” chapter for iOS #690

Merged
merged 1 commit into from
Feb 9, 2017
Merged

Conversation

sergey-akhalkov
Copy link
Contributor

@sergey-akhalkov sergey-akhalkov commented Feb 6, 2017

After RN 0.40 it is not possible to use other configurations than Debug and Release facebook/react-native#11813
As workaround we could make Staging’s Build Products Path value the same as for Release configuration.

Fix #688

@msftclas
Copy link

msftclas commented Feb 6, 2017

Hi @sergey-akhalkov, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!


It looks like you're a Microsoft contributor (Sergey Akhalkov (Akvelon)). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

@sergey-akhalkov sergey-akhalkov changed the title readme: improved “Multi-Deployment Testing” chapter for iOS [docs][ios] readme: improved “Multi-Deployment Testing” chapter for iOS Feb 7, 2017
Copy link
Contributor

@richardhuaaa richardhuaaa left a comment

Choose a reason for hiding this comment

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

Looks good! Just left a small comment, feel free to merge it after.

@@ -700,17 +700,21 @@ To set this up, perform the following steps:

6. Select the `Build Settings` tab

7. Click the `+` button on the toolbar and select `Add User-Defined Setting`
7. Go to `Build Location -> Per-configuration Build Products Path -> Staging` and change `Staging` value from `$(BUILD_DIR)/$(CONFIGURATION)$(EFFECTIVE_PLATFORM_NAME)` to `$(BUILD_DIR)/Release$(EFFECTIVE_PLATFORM_NAME)`
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you leave a short note here explaining why? (Similar to your PR description, with a link)

After RN 0.40 it is not possible to use other configurations than Debug and Release (facebook/react-native#11813)
As workaround we could make Staging’s Build Products Path value the same as for Release configuration.

Fix #688
@sergey-akhalkov
Copy link
Contributor Author

Fixed it, thanks!

@sergey-akhalkov sergey-akhalkov merged commit 04f618b into microsoft:master Feb 9, 2017
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.

3 participants