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

feat: Add CapWebView #5715

Merged
merged 24 commits into from
Jun 30, 2022
Merged

feat: Add CapWebView #5715

merged 24 commits into from
Jun 30, 2022

Conversation

jcesarmobile
Copy link
Member

No description provided.

elylucas and others added 22 commits September 27, 2021 13:46
…5487)

fix(ios): Add `getValue(_:)` method to JSValueContainer. This intends to replace `getAny(_:)`, since we are not ever going to be returning a type that does not conform to `JSValue`.

fix(ios): Update CAPWebView to use autolayout to pin the child webview to it's leading, trailing, top, and bottom constraints. This was causing issues with when CAPWebView is nested deep within a view heirarchy. Refactor to make `webView` and `capacitorBridge` to no longer be optionals through the use of lazy vars.
@Steven0351
Copy link
Member

One thing here that slipped my attention was the merge of main back into portals-dev. The native-publish branch references portals-dev for publishing to Cocoapods, which means all the future releases to Cocoapods will follow the 4.0.0 releases. I think while native-publish exists for iOS we should probably have portals-dev track the 3.x branch and if any changes need to go into portals-dev pre 4.0.0 general release we should cherry pick them into main as well.

@jcesarmobile
Copy link
Member Author

The idea of merging portals-dev into main is to get rid of portals-dev branch and with that the required extra step on publishing.

native-publish branch will always exist for publishing, we will need to update it to use main instead of portals-dev branch as portals-dev branch would be deleted after the merge of this PR.

If we are going to keep publishing v3 versions, we will need a new native-publish-v3 branch that points to the 3.x branch and cherry pick this CapWebView changes into 3.x branch too.

@jcesarmobile jcesarmobile merged commit 143d266 into main Jun 30, 2022
@jcesarmobile jcesarmobile deleted the portals-dev branch June 30, 2022 14:16
jcesarmobile added a commit that referenced this pull request Jun 30, 2022
Co-authored-by: Ely Lucas <ely@meta-tek.net>
Co-authored-by: Steven Sherry <steven.r.sherry@gmail.com>
Co-authored-by: Steven Sherry <steven.sherry@ionic.io>
@jcesarmobile jcesarmobile restored the portals-dev branch June 30, 2022 14:25
jcesarmobile added a commit that referenced this pull request Jul 1, 2022

Co-authored-by: Ely Lucas <ely@meta-tek.net>
Co-authored-by: Steven Sherry <steven.r.sherry@gmail.com>
Co-authored-by: Steven Sherry <steven.sherry@ionic.io>
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.

None yet

4 participants