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 objective c import in api.md #134

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

thorgas
Copy link

@thorgas thorgas commented Jun 8, 2023

hi @maxkomarychev,
thank you very much for this library! Not sure if something in my setup is wrong, but in order to use the env variables in Objective-c I had to change the import from your docs (https://github.com/maxkomarychev/react-native-ultimate-config/blob/master/docs/api.md#objective-c) to #import <react_native_ultimate_config/ConfigValues.h>

@maxkomarychev
Copy link
Owner

Hi, I have not experienced this. Could you please share more information about your setup? (version of Xcode, iOS deployment target, whether swift is used or no etc).

Thanks!

@thorgas
Copy link
Author

thorgas commented Sep 4, 2023

Hi,
my Xcode version is Version 14.3.1 (14E300c), IPHONEOS_DEPLOYMENT_TARGET = 14.0, we are using a bridging header and we have a module integrated via spm via Frameworks, Libraries, and embedded Content in Xcode under TARGETS -> General. RN version is 0.71.0. Above mentioned import happens in AppDelegate.mm

@dkulakov
Copy link

Hi @thorgas , thanks for your PR, that's really helpful for me!

@maxkomarychev this PR is fix for the issue #144

@alexkev
Copy link

alexkev commented Apr 30, 2024

Thanks @thorgas for your PR!

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.

4 participants