Skip to content

expensify-common hash update#4349

Merged
AndrewGable merged 5 commits intoExpensify:mainfrom
mateusbra:updated-expensify-common-hash
Aug 3, 2021
Merged

expensify-common hash update#4349
AndrewGable merged 5 commits intoExpensify:mainfrom
mateusbra:updated-expensify-common-hash

Conversation

@mateusbra
Copy link
Contributor

@AndrewGable will you please review this?

Hash update of expensify-common

Creation of html -> markdown conversion of inlineCodeBlock

Fixed Issues
#4185

Tests

WhatsApp.Video.2021-07-29.at.19.00.28.mp4

Unit test were added to tests/ExpensiMark-Markdown-test.js

QA

  1. Enter a message with inline code block
  2. Try to edit it
  3. Verify it will keep the inline code block markdown

@mateusbra mateusbra requested a review from a team as a code owner July 31, 2021 02:20
@MelvinBot MelvinBot requested review from AndrewGable and removed request for a team July 31, 2021 02:20
@github-actions
Copy link
Contributor

github-actions bot commented Jul 31, 2021

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@mateusbra
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@AndrewGable
Copy link
Contributor

AndrewGable commented Aug 2, 2021

Oops, looks like there is conflicts. Can you resolve them? Additionally, I think the commit should be Expensify/expensify-common@4016a78 (the merge commit hash)

@mateusbra
Copy link
Contributor Author

@AndrewGable I got some trouble solving the conflicts but I think now its ok and updated hash is ready for merging! 😅

@AndrewGable
Copy link
Contributor

I will merge once e2e tests pass (probably tomorrow morning!)

@AndrewGable AndrewGable merged commit dea2ec8 into Expensify:main Aug 3, 2021
@OSBotify
Copy link
Contributor

OSBotify commented Aug 3, 2021

✋ 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
Contributor

OSBotify commented Aug 6, 2021

🚀 Deployed to staging in version: 1.0.82-8🚀

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

@OSBotify
Copy link
Contributor

🚀 Deployed to production by @francoisl in version: 1.0.83-1 🚀

platform result
🤖 android 🤖 success ✅
🖥 desktop 🖥 failure ❌
🍎 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