Skip to content

Conversation

@lposen
Copy link
Collaborator

@lposen lposen commented May 3, 2025

🎟️ JIRA ticket(s)

πŸ• Description

Removed apiKey from the props to address concern from security

πŸ“· Screenshots

n/a

🧐 Testing

  1. In the root directory, run yarn prepare
  2. In the example directory:
    1. Create a .env.local file
    2. Copy the contents from .env into .env.local
    3. Replace YOUR_ITERABLE_API_KEY with an Iterable API Key
    4. Run npx expo prebuild --clean
    5. Run yarn ios to test ios
    6. Run yarn android to test android

The app should build and run as expected

πŸ“ Documentation

n/a

@lposen lposen changed the title MOB-11360-remove-apikey [MOB-11360] remove-apikey May 7, 2025


<!-- @import "[TOC]" {cmd="toc" depthFrom=2 depthTo=3 orderedList=false} -->

Copy link
Contributor

Choose a reason for hiding this comment

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

Seems like there is extra white space. Is this intentional?

},
"dependencies": {
"@iterable/react-native-sdk": "^2.0.0-beta",
"@iterable/react-native-sdk": "^2.0.0-beta.1",
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldnt this be 2.0.0-beta? 2.0.0-beta is the local verdaccio version right?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It should be 2.0.0-beta.1. There are circular dependencies in 2.0.0-beta which make it fail when imported into expo, though it works in the example app for some reason.

Copy link
Contributor

@evantk91 evantk91 left a comment

Choose a reason for hiding this comment

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

Left some comments.

</Text>
<Text style={styles.warningText}>
See the
<Text style={styles.emphasisTest}> example README</Text> or the
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice add!

@lposen lposen merged commit bfc7739 into main May 8, 2025
3 checks passed
@lposen lposen deleted the MOB-11360-remove-apikey branch May 8, 2025 19:09
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