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 min hight constant typo #9129

Merged
merged 3 commits into from
May 23, 2022
Merged

Fix min hight constant typo #9129

merged 3 commits into from
May 23, 2022

Conversation

Julesssss
Copy link
Contributor

Details

Fixes a small issue with this PR

Fixed Issues

$ #9128

Tests

QA Steps

For ipad/tablets*

For other platforms

Screenshots

As this was a hotfix to this PR I only built to web and iPad (the platform which was broken)

Web

  1. Screenshot 2022-05-23 at 17 07 12
  2. Screenshot 2022-05-23 at 17 07 05

iOS

Simulator Screen Shot - iPad Pro (12 9-inch) (5th generation) - 2022-05-23 at 17 31 07

@Julesssss Julesssss requested a review from a team as a code owner May 23, 2022 16:37
@Julesssss Julesssss self-assigned this May 23, 2022
@melvin-bot melvin-bot bot requested review from amyevans and removed request for a team May 23, 2022 16:38
@Julesssss Julesssss mentioned this pull request May 23, 2022
89 tasks
@Santhosh-Sellavel
Copy link
Collaborator

@Julesssss can we do one more correction here.

@Julesssss
Copy link
Contributor Author

@Julesssss can we do one more correction here.

Oh great, what is it?

@Santhosh-Sellavel
Copy link
Collaborator

Oh, you got that comment too, thanks! @Julesssss

@github-actions
Copy link
Contributor

⚠️ ⚠️ Heads up! This pull request has the CP Staging label. ⚠️ ⚠️
Merging it will cause it to be immediately deployed to staging, even if the open StagingDeployCash deploy checklist is locked.

@Julesssss
Copy link
Contributor Author

Nearly forgot theCherrypick label 😅

@amyevans amyevans merged commit a0c8daf into main May 23, 2022
@amyevans amyevans deleted the jules-fixMinHeightRegression branch May 23, 2022 16:51
OSBotify pushed a commit that referenced this pull request May 23, 2022
Fix min hight constant typo

(cherry picked from commit a0c8daf)
OSBotify added a commit that referenced this pull request May 23, 2022
@OSBotify
Copy link
Contributor

🚀 Cherry-picked to staging by @amyevans in version: 1.1.65-6 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

@Julesssss
Copy link
Contributor Author

Morning @Expensify/applauseleads, this one is ready for retesting now. Build 1.1.65-6

@mvtglobally
Copy link

@Julesssss #8752 (comment)

@Julesssss
Copy link
Contributor Author

Thanks, that looks good to me. Going to check it off.

@OSBotify
Copy link
Contributor

🚀 Deployed to production by @Julesssss in version: 1.1.65-6 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 success ✅
🍎 iOS 🍎 success ✅
🕸 web 🕸 success ✅

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.

6 participants