Skip to content

remove iOS container#67

Merged
liulianci merged 3 commits into
electrode-io:masterfrom
liulianci:scriptToAddContainer
Jan 11, 2020
Merged

remove iOS container#67
liulianci merged 3 commits into
electrode-io:masterfrom
liulianci:scriptToAddContainer

Conversation

@liulianci

Copy link
Copy Markdown
Contributor

added a script to create container. run ./ios/createContainer.sh to get iOS container

@friederbluemle friederbluemle left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Awesome! 👍

The new script does not show up in the diff (too many changes). Can you please either use two separate commits, or just paste the script to the PR description for review?

Comment thread .gitignore Outdated
Comment thread README.md Outdated
@liulianci
liulianci force-pushed the scriptToAddContainer branch from 174c98a to 4a65637 Compare January 2, 2020 22:18
@deepueg

deepueg commented Jan 6, 2020

Copy link
Copy Markdown
Contributor

Thanks @liulianci

Comment thread ios/createContainer.sh Outdated
@liulianci
liulianci force-pushed the scriptToAddContainer branch 5 times, most recently from 057fd1f to 044d0f6 Compare January 10, 2020 21:10
@liulianci

Copy link
Copy Markdown
Contributor Author

This is part of PR #60

@liulianci
liulianci force-pushed the scriptToAddContainer branch from 044d0f6 to d0a6e2c Compare January 10, 2020 21:13
add ElectrodeContainer to embed frameworks to fix a crash on Device:
dyld: Library not loaded: @rpath/ElectrodeContainer.framework/ElectrodeContainer

@friederbluemle friederbluemle left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

@liulianci
liulianci merged commit 9cc9499 into electrode-io:master Jan 11, 2020
@liulianci
liulianci deleted the scriptToAddContainer branch January 11, 2020 00:46
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