Skip to content

[No QA] Add clarification notes on datepicker#5627

Merged
chiragsalian merged 1 commit into
Expensify:mainfrom
kidroca:kidroca/datepicker
Oct 4, 2021
Merged

[No QA] Add clarification notes on datepicker#5627
chiragsalian merged 1 commit into
Expensify:mainfrom
kidroca:kidroca/datepicker

Conversation

@kidroca

@kidroca kidroca commented Oct 4, 2021

Copy link
Copy Markdown
Contributor

Details

Add documentation notes regarding

Fixed Issues

$ N/A

Tests

Only added comments
Tested that the app still works

QA Steps

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

cc @roryabraham

@kidroca kidroca requested a review from a team as a code owner October 4, 2021 14:20
@MelvinBot MelvinBot requested review from chiragsalian and removed request for a team October 4, 2021 14:20

@chiragsalian chiragsalian 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.

LGTM

@chiragsalian chiragsalian merged commit 1e70165 into Expensify:main Oct 4, 2021
@OSBotify

OSBotify commented Oct 4, 2021

Copy link
Copy Markdown
Contributor

🚀 Deployed to staging by @chiragsalian in version: 1.1.4-2 🚀

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

@OSBotify

OSBotify commented Oct 6, 2021

Copy link
Copy Markdown
Contributor

🚀 Deployed to production by @chiragsalian in version: 1.1.5-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