Skip to content

pointer on top within close account page#7498

Merged
iwiznia merged 1 commit into
Expensify:mainfrom
mateusbra:pointer-on-top
Feb 7, 2022
Merged

pointer on top within close account page#7498
iwiznia merged 1 commit into
Expensify:mainfrom
mateusbra:pointer-on-top

Conversation

@mateusbra

@mateusbra mateusbra commented Feb 1, 2022

Copy link
Copy Markdown
Contributor

Details

Style change for pointer to start on top on android on close account page

Fixed Issues

$ #7380

Tests | QA Steps

  1. login with new expensify account
  2. go to settings
  3. go to security page
  4. go to close account page
  5. verify that the pointer starts on top

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

image

Mobile Web

image

Desktop

Captura de Tela 2022-02-01 às 09 49 49

iOS

Captura de Tela 2022-02-01 às 13 02 21

Android

image

@mateusbra mateusbra requested a review from a team as a code owner February 1, 2022 10:34
@MelvinBot MelvinBot requested review from iwiznia and parasharrajat and removed request for a team February 1, 2022 10:34

@parasharrajat parasharrajat left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Change looks good. But please fix the template
image

Then test it on all platforms and add the screenshots. We are on the verge of making this mandatory for all PRs.

@mateusbra

Copy link
Copy Markdown
Contributor Author

@parasharrajat tested on all platforms and fixed the template

@parasharrajat parasharrajat left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM.

cc: @iwiznia

🎀 👀 🎀 C+ reviewed

@mateusbra

Copy link
Copy Markdown
Contributor Author

@iwiznia do you think we have to change something, or its ok for this PR?

@iwiznia iwiznia merged commit 37e3e57 into Expensify:main Feb 7, 2022
@OSBotify

OSBotify commented Feb 7, 2022

Copy link
Copy Markdown
Contributor

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@OSBotify

OSBotify commented Feb 9, 2022

Copy link
Copy Markdown
Contributor

🚀 Deployed to staging by @iwiznia in version: 1.1.38-0 🚀

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

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to production by @Julesssss in version: 1.1.38-3 🚀

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.

4 participants