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(camera-plus): fixes outstanding issues #35

Merged
merged 1 commit into from
Jul 3, 2021

Conversation

Logikgate
Copy link
Contributor

fixes #12

The first issue in #12 was fixed by NathanWalker/SwiftyCam#10.

The second issue is fixed by this pull request, the methods needed to be exposed to objective-c so they could be called natively.

Leaving this as a WIP for now because it relies on NativeScript/ios#116 being merged and a new version of the runtime being released.

@NathanWalker
Copy link
Member

Thank you for this @Logikgate and sorry for delay on processing. We are going to bring several patches together on ios runtime including your PR and publish another 'rc' to validate this changeset against and will try to get a patch published next week on this.

@NathanWalker
Copy link
Member

@nativescript/ios rc published now containing NativeScript/ios#116 - we are merging this into a release staging branch to confirm.

@NathanWalker NathanWalker marked this pull request as ready for review July 3, 2021 20:05
@NathanWalker NathanWalker changed the base branch from master to release/camplus July 3, 2021 20:05
@NathanWalker NathanWalker merged commit ab3e00b into nstudio:release/camplus Jul 3, 2021
@NathanWalker
Copy link
Member

NathanWalker commented Jul 3, 2021

@Logikgate testing this and appears a crash still occurs. It's interesting as it seems to crash on just the previewLayer accessor, I added the info here:
0057a92

Not sure if you had seen anything similar at anytime?

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.

[camera-plus] Known issues: video recording and preview layer.
2 participants