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

chore(bumped-ecommon-ver): Updated expensify-common version #4295

Merged
merged 1 commit into from
Jul 29, 2021

Conversation

mananjadhav
Copy link
Collaborator

@AndrewGable Hopefully this should work!

Details

  • Merges hyperlink changes from expensify-common to the App

Fixed Issues

$ Fixes App/4229

Tests

QA Steps

  1. Type [Yo (click here to see a cool cat)](https://c8.alamy.com/compes/ha11pc/cookie-cat-con-sombrero-de-cowboy-y-sun-glass-ha11pc.jpg) (without `)
  2. Verify it hyperlinks correctly

Tested On

  • Web
  • Mobile Web
  • Desktop
  • iOS
  • Android

Screenshots

Web

md-hyperlink-web-screencast.mp4

Mobile Web

md-hyperlink-mobile-web-screencast.mp4

Desktop

md-hyperlink-desktop-screencast.mp4

iOS

md-hyperlink-ios-screencast.mp4

Android

md-hyperlink-android-screencast.mp4

@mananjadhav mananjadhav requested a review from a team as a code owner July 29, 2021 03:17
@MelvinBot MelvinBot requested review from Luke9389 and removed request for a team July 29, 2021 03:18
@AndrewGable
Copy link
Contributor

Ok seeing all the checks running now 🥳 Thank you for your patience on this one!

@AndrewGable
Copy link
Contributor

@Luke9389 - This is @mananjadhav's third try at this same PR, so I've reviewed the rest of this code. It looks great, so I am going to merge before your review 🚀

@AndrewGable AndrewGable merged commit d26de05 into Expensify:main Jul 29, 2021
@OSBotify
Copy link
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.

@mananjadhav
Copy link
Collaborator Author

Finally. Thanks for merging. Took more effort to merge the PR than to solve the bug 😝

@AndrewGable
Copy link
Contributor

Sorry about that @mananjadhav - Hopefully your next few PRs will be more smooth 😆

@OSBotify
Copy link
Contributor

🚀 Deployed to staging in version: 1.0.81-5🚀

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

@OSBotify
Copy link
Contributor

OSBotify commented Aug 6, 2021

🚀 Deployed to production in version: 1.0.82-7🚀

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.

Hyperlink markdown doesn't work if the ] is preceded by an (
3 participants