Skip to content

Preserve the whitespace in the mesages#6975

Merged
deetergp merged 1 commit into
Expensify:mainfrom
parasharrajat:whitespace-preserve
Jan 5, 2022
Merged

Preserve the whitespace in the mesages#6975
deetergp merged 1 commit into
Expensify:mainfrom
parasharrajat:whitespace-preserve

Conversation

@parasharrajat

@parasharrajat parasharrajat commented Jan 1, 2022

Copy link
Copy Markdown
Member

Details

Fixed Issues

$ #6769

Tests | QA Steps

  1. Send some message which has whitespace.
  2. Whitespace should be preserved.

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web | Desktop

image

Mobile Web

Screenshot 2022-01-04 22:34:19

iOS

Android

Screenshot 2022-01-04 23:18:02

@parasharrajat parasharrajat marked this pull request as ready for review January 5, 2022 12:19
@parasharrajat parasharrajat requested a review from a team as a code owner January 5, 2022 12:19
@botify botify requested a review from deetergp January 5, 2022 12:19
@MelvinBot MelvinBot removed the request for review from a team January 5, 2022 12:19

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

Passes tests; LGTM!

@deetergp deetergp merged commit 6f97362 into Expensify:main Jan 5, 2022
@OSBotify

OSBotify commented Jan 5, 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 Jan 6, 2022

Copy link
Copy Markdown
Contributor

🚀 Deployed to staging by @deetergp in version: 1.1.25-2 🚀

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

@OSBotify

Copy link
Copy Markdown
Contributor

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