Skip to content

[MOO-1344] add mendix native iOS files - #2

Merged
stelselim merged 11 commits into
mainfrom
ios-implementation
Mar 13, 2024
Merged

[MOO-1344] add mendix native iOS files#2
stelselim merged 11 commits into
mainfrom
ios-implementation

Conversation

@stelselim

@stelselim stelselim commented Feb 23, 2024

Copy link
Copy Markdown
Collaborator
  • mendix native library iOS source code added.
  • Required dependencies added to Podfile of example project.

@stelselim stelselim changed the title Ios implementation [MOO-1344] add mendix native iOS files Feb 27, 2024
@stelselim stelselim self-assigned this Feb 27, 2024
Comment thread .gitignore
*.ipa
*.xcuserstate
project.xcworkspace
*.env.local

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Might not be a bad idea to commit this since it's most likely going to be identical for everyone, WDYT?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

It was necessary for example project. It was using the my local path which is unique to my device.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Oh, then it's fine to ignore. If you were to generate a blank RN app, they will add the file for you and the contents should work for everyone, it's something like "export NODE_PATH=command -v node". We can add it later though.

#import "React/RCTRedBox.h"
#import "ReactNative.h"

// Used by previous versions of the client (<= 9.15)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Can you please create a jira issue to see if we can remove this?

@vadymv-mendix vadymv-mendix left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

👍

@stelselim
stelselim force-pushed the ios-implementation branch from c951409 to 7178b18 Compare March 13, 2024 13:32
@stelselim
stelselim merged commit 6a82002 into main Mar 13, 2024
@UrazAkgultan
UrazAkgultan deleted the ios-implementation branch March 13, 2024 15:13
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