-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
expensify-common hash update #4349
Conversation
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
I have read the CLA Document and I hereby sign the CLA |
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) |
@AndrewGable I got some trouble solving the conflicts but I think now its ok and updated hash is ready for merging! 😅 |
I will merge once e2e tests pass (probably tomorrow morning!) |
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
🚀 Deployed to staging in version: 1.0.82-8🚀
|
🚀 Deployed to production by @francoisl in version: 1.0.83-1 🚀
|
@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