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

Fix react-native 0.53 prompt glitches when linking #252

Merged
merged 4 commits into from
Feb 28, 2018

Conversation

guperrot
Copy link
Member

No description provided.

@guperrot guperrot requested a review from dhei February 28, 2018 21:51
value: 'ASK_JAVASCRIPT'
}]
}]).catch((e) => {
console.log(`Could not determine when to enable AppCenter crashes. Error Reason - ${e.message}`);
Copy link
Member

@dhei dhei Feb 28, 2018

Choose a reason for hiding this comment

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

The original log message Could not determine when to send AppCenter crashes. sounds better to me, it's more accurate and informative.

Copy link
Member Author

@guperrot guperrot Feb 28, 2018

Choose a reason for hiding this comment

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

Sounds like we didn't have the same message as analytics then, I copied from there.

value: 'ASK_JAVASCRIPT'
}]
}]).catch((e) => {
console.log(`Could not determine when to enable AppCenter crashes. Error Reason - ${e.message}`);
Copy link
Member

Choose a reason for hiding this comment

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

same as above comment.

Copy link
Member

@dhei dhei left a comment

Choose a reason for hiding this comment

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

Can you please make sure eslint.sh linting passes for your PR. Other than that, PR looks good to me.

@dhei dhei merged commit 1a2ab3a into develop Feb 28, 2018
@guperrot guperrot deleted the fix/link-prompts-rn053 branch March 1, 2018 00:49
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.

2 participants