Skip to content

feat: ability to embed into platform host projects - #1799

Open
vmutafov wants to merge 6 commits into
NativeScript:mainfrom
vmutafov:embed
Open

feat: ability to embed into platform host projects#1799
vmutafov wants to merge 6 commits into
NativeScript:mainfrom
vmutafov:embed

Conversation

@vmutafov

Copy link
Copy Markdown
Contributor

No description provided.

@cla-botcla-botBot added the cla: yes label Jan 2, 2024
@edusperoni
edusperoni changed the base branch from embed to mainApril 2, 2024 16:17

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.

This file should be reverted

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.

revert to keep history clean

Comment on lines +182 to +185
implementation "com.google.android.material:material:$androidXMaterialVersion"
implementation "androidx.appcompat:appcompat:$androidXAppCompatVersion"
implementation 'androidx.navigation:navigation-fragment-ktx:2.5.3'
implementation 'androidx.navigation:navigation-ui-ktx:2.5.3'

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.

are these needed? It'd mean all apps now depend on material, androidx navigation and appcompat

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.

This file might need to be gitignored (NativeScriptActivity.java isn't currently committed into main)

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.

This file might need to be gitignored (NativeScriptApplication.java isn't currently committed into main)

@NathanWalkerNathanWalker changed the title feat: add android embedding and merge mainfeat: ability to embed into platform host projectsJul 2, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@vmutafov@edusperoni