-
Notifications
You must be signed in to change notification settings - Fork 3.2k
[HOLD for payment 2022-09-26] [$1000] When you send old :expensify: link and hover you will not see any tooltip and the link is without underline - reported by @Puneet-here #9739
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
Comments
Triggered auto assignment to @muttmuure ( |
Triggered auto assignment to @mountiny ( |
Triggered auto assignment to @puneetlath ( |
This definitely seems as a suitable External issue |
Upwork job is here: https://www.upwork.com/jobs/~0123b2819ac262a34e |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @parasharrajat ( |
Current assignee @puneetlath is eligible for the Exported assigner, not assigning anyone new. |
A request, can any one provide old and new link in below comment ? |
@parasharrajat @mananjadhav I've sent you both hiring offers through Upwork. Please accept when you get a chance so that we can pay you out. Thanks! |
Let's hold the payment, there is a issue in my last PR implementation. I have already created a PR with the fix that needs to be reviewed. |
This comment was marked as outdated.
This comment was marked as outdated.
@puneetlath, @mananjadhav, @parasharrajat Eep! 4 days overdue now. Issues have feelings too... |
Not yet ready for payment. There is another PR waiting to be reviewed. |
@puneetlath, @mananjadhav, @parasharrajat Whoops! This issue is 2 days overdue. Let's get this updated quick! |
@puneetlath, @mananjadhav, @parasharrajat Whoops! This issue is 2 days overdue. Let's get this updated quick! |
@puneetlath, @mananjadhav, @parasharrajat 6 days overdue. This is scarier than being forced to listen to Vogon poetry! |
PR is awaiting review. #10503 |
Looks like that PR was merged yesterday. Will pay this out in 6 days if no more regressions. |
@puneetlath, @mananjadhav, @parasharrajat Whoops! This issue is 2 days overdue. Let's get this updated quick! |
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.2.1-0 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2022-09-26. 🎊 |
@mananjadhav - paid. |
Applied! Thanks. |
All paid! Thanks everyone. |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Action Performed:
Expected Result:
You should see underline below the link and tooltip when you hover
Actual Result:
You don't see any tooltip or underline
Workaround:
unknown
Platform:
Where is this issue occurring?
Version Number: 1.1.79-11
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Screen.Recording.2022-06-25.at.12.41.08.AM.mov
Expensify/Expensify Issue URL:
Issue reported by: @Puneet-here
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1656098692618689
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: