-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Settings - Share code - Share button is non functional #19041
Comments
Triggered auto assignment to @kadiealexander ( |
Bug0 Triage Checklist (Main S/O)
|
This is not a bug, It actually copies the link to clipboard, perhaps it shouldn't be called Share or, maybe a feedback message saying the link has been copied to clipboard? or maybe it's called Share because the authors are planning to change the functionality in future. |
According to a slack discussion here, I think the plan is to go with changing the arrow icon with copy to clipboard icon. |
This issue seems to be fixed by this PR. But it is not appropriately linked to the fixing PR. Linking it to the fixing PR might be a good idea to track the progress of this report. Here is the slack report of this GH issue for more context |
Thanks for flagging that PR @Natnael-Guchima, this behaviour is now fixed: 2023-05-19_13-12-46.mp4Closing this issue! |
You are welcome @kadiealexander. I am not paid for the original reporting though. I think the discussion to change the icon from an arrow icon to copy clipboard was first triggered by my report here https://expensify.slack.com/archives/C049HHMV9SM/p1684154588458859?thread_ts=1684154588.458859&cid=C049HHMV9SM. You can go through the report and the replies for evidence. Thanks. |
@kadiealexander yep I think @Natnael-Guchima was first to note this bug so we should pay them for reporting it. Reopening to handle the payment. thank you! |
Job added to Upwork: https://www.upwork.com/jobs/~0154212e434bfba80a |
Triggered auto assignment to Contributor Plus for review of internal employee PR - @mollfpr ( |
Added the internal label to spin up an Upwork job. |
@Natnael-Guchima sent you an offer, please let me know when you accept. |
I have accepted the offer, @kadiealexander. Thanks. |
Paid! Thanks @Natnael-Guchima |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Issue found when executing PR #18636
Action Performed:
Expected Result:
Share button is functional
Actual Result:
Share button is non functional
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.3.14.4
Reproducible in staging?: Yes
Reproducible in production?: New feature
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team and @Natnael-Guchima
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: