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

Allow overriding api key in RN iOS via event callback #980

Merged
merged 2 commits into from Aug 4, 2020

Conversation

fractalwrench
Copy link
Contributor

Goal

Allows overriding the API key in React Native iOS in an event callback by reading the value passed down from the JS layer. This only applies to the iOS layer for now.

Added an E2E scenario to cover the change.

@bugsnagbot
Copy link
Collaborator

bugsnagbot commented Aug 4, 2020

@bugsnag/browser bundle size diff

Minified Minfied + Gzipped
Before 39.63 kB 12.24 kB
After 39.63 kB 12.24 kB
± No change No change

Generated by 🚫 dangerJS against a03c468

Copy link
Contributor

@bengourley bengourley 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. Couple of things I think can be removed.

Co-authored-by: Ben Gourley <bn@grly.me>
@fractalwrench fractalwrench merged commit 96f2e6d into v7-react-native Aug 4, 2020
@fractalwrench fractalwrench deleted the v7rn-ios-api-key branch August 4, 2020 15:53
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.

None yet

3 participants