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

[No QA] Changed App Store Icon #4048

Merged
merged 1 commit into from
Jul 14, 2021
Merged

[No QA] Changed App Store Icon #4048

merged 1 commit into from
Jul 14, 2021

Conversation

Luke9389
Copy link
Contributor

Details

We were getting a deployment error due to a transparent icon, and we believe we've replaced the culprit 🤞

Fixed Issues

Slack thread: https://expensify.slack.com/archives/C01GTK53T8Q/p1626293141260600

Tests

QA Steps

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

@Luke9389 Luke9389 self-assigned this Jul 14, 2021
@Luke9389 Luke9389 requested a review from a team as a code owner July 14, 2021 20:52
@MelvinBot MelvinBot requested review from MonilBhavsar and removed request for a team July 14, 2021 20:52
@OSBotify
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.

@AndrewGable AndrewGable merged commit 37d013e into main Jul 14, 2021
@AndrewGable AndrewGable deleted the luke-new-app-store-icon branch July 14, 2021 21:52
@OSBotify
Copy link
Contributor

🚀 Cherry-picked to staging in version: 1.0.77-7🚀

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

@OSBotify
Copy link
Contributor

🚀 Deployed to production in version: 1.0.79-4🚀

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.

3 participants