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

Instant Debits: Polished the flow #3590

Merged
merged 1 commit into from
May 28, 2024
Merged

Conversation

kgaidis-stripe
Copy link
Contributor

@kgaidis-stripe kgaidis-stripe commented May 17, 2024

Summary

^ small code polish around Instant Debits (both code and non-code)

Testing

Start Flow

Before After
Simulator Screenshot - iPhone 15 Pro - 2024-05-24 at 15 10 12 Simulator Screenshot - iPhone 15 Pro - 2024-05-24 at 15 03 44

End Flow

Before After
Simulator Screenshot - iPhone 15 Pro - 2024-05-24 at 15 10 30 Simulator Screenshot - iPhone 15 Pro - 2024-05-24 at 15 04 06

Copy link

emerge-tools bot commented May 17, 2024

1 build increased size, 3 builds decreased size, 2 builds had no size change

Name Version Download Change Install Change Approval
StripePaymentsUISize
com.stripe.StripePaymentsUISize
1.0 (1) 1.8 MB ⬆️ 147 B 6.1 MB ⬇️ 444 B N/A
StripeFinancialConnectionsSize
com.stripe.StripeFinancialConnectionsSize
1.0 (1) 1.2 MB - 3.9 MB - N/A
StripePaymentsSize
com.stripe.StripePaymentsSize
1.0 (1) 1.1 MB ⬇️ 83 B 3.9 MB ⬇️ 444 B (-0.01%) N/A
StripePaymentSheetSize
com.stripe.StripePaymentSheetSize
1.0 (1) 3.2 MB ⬆️ 1.4 kB (0.04%) 9.6 MB ⬆️ 1.3 kB (0.01%) N/A
StripeApplePaySize
com.stripe.StripeApplePaySize
1.0 (1) 422.4 kB - 1.5 MB - N/A
StripeSize
com.stripe.StripeSize
1.0 (1) 2.3 MB ⬇️ 63 B 7.6 MB ⬇️ 444 B N/A

StripePaymentsUISize 1.0 (1)
com.stripe.StripePaymentsUISize

⚖️ Compare build
⏱️ Analyze build performance

Total install size change: ⬇️ 444 B
Total download size change: ⬆️ 147 B

Largest size changes

Item Install Size Change
Other ⬇️ -444 B
View Treemap

Image of diff

StripeFinancialConnectionsSize 1.0 (1)
com.stripe.StripeFinancialConnectionsSize

No changes to report

StripePaymentsSize 1.0 (1)
com.stripe.StripePaymentsSize

⚖️ Compare build
⏱️ Analyze build performance

Total install size change: ⬇️ 444 B (-0.01%)
Total download size change: ⬇️ 83 B

Largest size changes

Item Install Size Change
Other ⬇️ -444 B
View Treemap

Image of diff

StripePaymentSheetSize 1.0 (1)
com.stripe.StripePaymentSheetSize

⚖️ Compare build
⏱️ Analyze build performance

Total install size change: ⬆️ 1.3 kB (0.01%)
Total download size change: ⬆️ 1.4 kB (0.04%)

Largest size changes

Item Install Size Change
StripePaymentSheet.PaymentSheetFormFactory.makeUSBankAccount(merc... ⬆️ 1.2 kB
StripePaymentSheet.PaymentSheetFormFactory.makeInstantDebits ⬆️ 588 B
Other ⬇️ -408 B
View Treemap

Image of diff

StripeApplePaySize 1.0 (1)
com.stripe.StripeApplePaySize

No changes to report

StripeSize 1.0 (1)
com.stripe.StripeSize

⚖️ Compare build
⏱️ Analyze build performance

Total install size change: ⬇️ 444 B
Total download size change: ⬇️ 63 B

Largest size changes

Item Install Size Change
Other ⬇️ -444 B
View Treemap

Image of diff


🛸 Powered by Emerge Tools

Base automatically changed from kg-instadeb to master May 20, 2024 20:40
Copy link

emerge-tools bot commented May 20, 2024

📸 Snapshot Test

No snapshots generated

Name Version Added Removed Modified Unchanged Errored Approval
StripePaymentsUISize
com.stripe.StripePaymentsUISize
1.0 (1) 0 0 0 0 0 N/A
StripeFinancialConnectionsSize
com.stripe.StripeFinancialConnectionsSize
1.0 (1) 0 0 0 0 0 N/A
StripePaymentsSize
com.stripe.StripePaymentsSize
1.0 (1) 0 0 0 0 0 N/A
StripePaymentSheetSize
com.stripe.StripePaymentSheetSize
1.0 (1) 0 0 0 0 0 N/A
StripeApplePaySize
com.stripe.StripeApplePaySize
1.0 (1) 0 0 0 0 0 N/A
StripeSize
com.stripe.StripeSize
1.0 (1) 0 0 0 0 0 N/A

🛸 Powered by Emerge Tools

@kgaidis-stripe kgaidis-stripe changed the base branch from master to kg-instadeb-tests May 21, 2024 21:35
@stripe stripe deleted a comment from github-actions bot May 21, 2024
Base automatically changed from kg-instadeb-tests to master May 24, 2024 20:43
@kgaidis-stripe kgaidis-stripe marked this pull request as ready for review May 24, 2024 22:12
@kgaidis-stripe kgaidis-stripe requested review from a team as code owners May 24, 2024 22:12
@kgaidis-stripe kgaidis-stripe merged commit e9adfe5 into master May 28, 2024
5 checks passed
@kgaidis-stripe kgaidis-stripe deleted the kg-instadeb-polish branch May 28, 2024 16:24
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

2 participants