-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
[$62.50] The filter bar on Search page contains the wrong icons when set to Chat #57763
Comments
Triggered auto assignment to @greg-schroeder ( |
Proposal by @shubham1206agra ProposalPlease re-state the problem that we are trying to solve in this issue.The filter bar on Search page contains the wrong icons when set to Chat What is the root cause of that problem?Design Change Request What changes do you think we should make in order to solve the problem?Change icons in following places to whatever is required in the issue
What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?Not Required as this is a simple icon change What alternative solutions did you explore? (Optional) |
Job added to Upwork: https://www.upwork.com/jobs/~021897406334382928419 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @brunovjk ( |
I'm working on some PRs review now, I'm going to review the proposal ASAP. Thank you. |
ProposalPlease re-state the problem that we are trying to solve in this issue.The filter bar on Search page contains the wrong icons when set to Chat What is the root cause of that problem?Wrong icon assigned to "Attachments" and "Links" filter options What changes do you think we should make in order to solve the problem?Change the icons as following: App/src/components/Search/SearchPageHeader/SearchStatusBar.tsx Lines 128 to 131 in b649380
to icon: Expensicons.Paperclip
App/src/components/Search/SearchPageHeader/SearchStatusBar.tsx Lines 134 to 137 in b649380
to icon: Expensicons.Link
What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?None What alternative solutions did you explore? (Optional)None |
@shubham1206agra's proposal looks fine to me. I wonder if there are other icons we should check, as this bug seems to have been in the prod for some time, I haven't found any yet. 🎀👀🎀 C+ reviewed |
Triggered auto assignment to @roryabraham, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
@brunovjk Whoops! This issue is 2 days overdue. Let's get this updated quick! |
Friendly bump @roryabraham on Slack |
📣 @brunovjk 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app! |
just wanted to clarify that the price for this issue is intended to be $62.50, not $250 |
@roryabraham Can you keep this at $100? |
@Expensify/design Can you send the attachment icon? |
The icons are here:
|
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 9.1.8-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
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
Expensify/Expensify Issue URL:
Issue reported by: @dubielzyk-expensify
Slack conversation (hyperlinked to channel name): #Expensify Bugs
Action Performed:
Expected Result:
The icons should be as outlined in the design document
Actual Result:
The icons are not as per the design document
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @brunovjkThe text was updated successfully, but these errors were encountered: