Skip to content

[No QA] Cleanup - Remove keyboard type numeric, and incorrect uses of phone-pad.#7287

Merged
deetergp merged 2 commits into
Expensify:mainfrom
rushatgabhane:main
Jan 18, 2022
Merged

[No QA] Cleanup - Remove keyboard type numeric, and incorrect uses of phone-pad.#7287
deetergp merged 2 commits into
Expensify:mainfrom
rushatgabhane:main

Conversation

@rushatgabhane

@rushatgabhane rushatgabhane commented Jan 18, 2022

Copy link
Copy Markdown
Member

This is a cleanup PR. Details here

  • numeric keyboard type is removed, and replaced with number-pad (because they're same)

  • Incorrect usage of phone-pad is replaced with number-pad. (phone-pad is only used for phone numbers.)

Fixed Issues

$ #7286

Tests

  • Verify that no errors appear in the JS console

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

No errors

Mobile Web

Desktop

No errors

iOS

Android

@rushatgabhane rushatgabhane requested a review from a team as a code owner January 18, 2022 01:17
@MelvinBot MelvinBot requested review from deetergp and removed request for a team January 18, 2022 01:18
@deetergp deetergp self-assigned this Jan 18, 2022

@deetergp deetergp left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me! :chefkiss:

@deetergp deetergp merged commit c330dcd into Expensify:main Jan 18, 2022
@OSBotify

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

Copy link
Copy Markdown
Contributor

🚀 Deployed to staging by @deetergp in version: 1.1.30-4 🚀

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

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Deployed to production by @chiragsalian in version: 1.1.31-1 🚀

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