Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

expensify-common hash update #4349

Merged
merged 5 commits into from
Aug 3, 2021

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