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

Add custom footer view support #792

Merged
merged 18 commits into from
Oct 16, 2017
Merged

Conversation

bdorfman-stripe
Copy link
Contributor

Taking over from #767

Adds the ability to provide a custom footer view that will be added to the view controllers managed by the Stripe SDK.

Remove the footer view supporting protocol because it didn't end up being that useful in the end.
Specifically separate out footer view properties for add card VC and payment methods VC.
Make sure if you set the prop before the view even loads we defer trying to attach it to the table view until the table exists and is sized correctly using viewWillAppear promise.
Copy link
Contributor

@bg-stripe bg-stripe left a comment

Choose a reason for hiding this comment

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

lgtm!

@bdorfman-stripe
Copy link
Contributor Author

Re-r with newest commit @bg-stripe ?

mludowise-stripe added a commit that referenced this pull request Mar 2, 2022
Optimizes the DocumentScanner to only perform up
to 2 concurrent scan requests at a time, forcing
the `AVCaptureSession` delegate to drop frames
when the scanner is busy processing images and
blocking the video capture thread. On most
devices, the ideal maximum number of concurrent
scan requests is 2 so we can take advantage of
parallelizing processing across both the CPU and
GPU.
Sign up for free to 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.

4 participants