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

[Unticketed] Updated Stripe Framework #1718

Merged
merged 1 commit into from Sep 1, 2022
Merged

Conversation

msadoon
Copy link
Contributor

@msadoon msadoon commented Sep 1, 2022

📲 What

The new Stripe framework fixes a bug where 4.7 inch screens launched in landscape do not appear unless rotated. Very minor.

🤔 Why

We would ideally like to support all orientations for all phones natively

🛠 How

Stripe upgraded their framework.

👀 See

Before 🐛

Simulator.Screen.Recording.-.iPhone.SE.3rd.generation.-.2022-09-01.at.14.10.17.mp4

After 🦋

Simulator.Screen.Recording.-.iPhone.SE.3rd.generation.-.2022-09-01.at.14.05.11.mp4

✅ Acceptance criteria

  • Launch the app on iPhone SE while in landscape (unrotated) ensure the

@msadoon msadoon added this to the release-5.5.0 milestone Sep 1, 2022
@msadoon msadoon self-assigned this Sep 1, 2022
@codecov
Copy link

codecov bot commented Sep 1, 2022

Codecov Report

Merging #1718 (03444da) into main (5304313) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1718   +/-   ##
=======================================
  Coverage   85.33%   85.33%           
=======================================
  Files        1275     1275           
  Lines      114806   114806           
  Branches    30343    30343           
=======================================
+ Hits        97968    97970    +2     
+ Misses      15786    15784    -2     
  Partials     1052     1052           
Impacted Files Coverage Δ
Library/Navigation.swift 85.85% <0.00%> (+0.64%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@msadoon msadoon merged commit 14d51d0 into main Sep 1, 2022
@msadoon msadoon deleted the fix/update-stripe-framework branch September 1, 2022 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants