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

[HOLD for payment 2024-09-06][Search v2.1] Add Filters button to Search page #47932

Closed
luacmartins opened this issue Aug 23, 2024 · 14 comments
Closed
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Daily KSv2 NewFeature Something to build that is a new item.

Comments

@luacmartins
Copy link
Contributor

luacmartins commented Aug 23, 2024

We should add the Filters button to the Search page header so users can navigate to the Filters page

Issue OwnerCurrent Issue Owner: @stephanieelliott
@luacmartins luacmartins added Daily KSv2 NewFeature Something to build that is a new item. labels Aug 23, 2024
Copy link

melvin-bot bot commented Aug 23, 2024

Triggered auto assignment to @stephanieelliott (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details. Please add this Feature request to a GH project, as outlined in the SO.

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Daily KSv2 labels Aug 23, 2024
Copy link

melvin-bot bot commented Aug 23, 2024

⚠️ It looks like this issue is labelled as a New Feature but not tied to any GitHub Project. Keep in mind that all new features should be tied to GitHub Projects in order to properly track external CAP software time ⚠️

Copy link

melvin-bot bot commented Aug 23, 2024

Triggered auto assignment to Design team member for new feature review - @dubielzyk-expensify (NewFeature)

@luacmartins luacmartins added the Reviewing Has a PR in review label Aug 23, 2024
@luacmartins luacmartins mentioned this issue Aug 23, 2024
50 tasks
@dubielzyk-expensify
Copy link
Contributor

Looks good on desktop:
CleanShot 2024-08-26 at 14 47 55@2x

This happens on mobile though:
You tap the filter button and you get a brief flash of the most recent chat before the filter screen appears:

CleanShot.2024-08-26.at.14.47.31.mp4

cc @Expensify/design

@shawnborton
Copy link
Contributor

Good catch.

Also is it just me, or are the two button background colors different from one another?
CleanShot 2024-08-26 at 07 11 18@2x

@shawnborton
Copy link
Contributor

(Actually we should probably leave this feedback in the PR itself)

@luacmartins
Copy link
Contributor Author

Hahah yea, leaving the comments on the PR would be great. The PR was merged earlier today.

You tap the filter button and you get a brief flash of the most recent chat before the filter screen appears:

I couldn't reproduce this locally. I'll try again in staging once this PR is deployed

Also is it just me, or are the two button background colors different from one another?

I see both with background-color: rgb(230, 225, 218) so they seem to match.

@rayane-djouah
Copy link
Contributor

rayane-djouah commented Aug 31, 2024

⚠️ We just got confirmation on Slack that the Deploy Checklist: New Expensify 2024-08-26 which includes the PR of this issue was only deployed to production in Deploy Checklist: New Expensify 2024-08-28. More context on why this happened can be found in this Slack thread and this Slack comment.

Given the context above, this issue should be on [HOLD for Payment 2024-09-6] according to production deploy from Deploy Checklist: New Expensify 2024-08-28.

cc @stephanieelliott

@dubielzyk-expensify
Copy link
Contributor

Yeah, even when I overlay them they seem to be the same color
CleanShot 2024-09-02 at 11 00 29@2x

@luacmartins luacmartins changed the title [Search v2.1] Add Filters button to Search page [HOLD for payment 2024-09-06][Search v2.1] Add Filters button to Search page Sep 2, 2024
@luacmartins luacmartins added Awaiting Payment Auto-added when associated PR is deployed to production Daily KSv2 and removed Reviewing Has a PR in review Weekly KSv2 labels Sep 2, 2024
@melvin-bot melvin-bot bot added the Overdue label Sep 4, 2024
@luacmartins
Copy link
Contributor Author

luacmartins commented Sep 4, 2024

Not overdue. Payment is due on the 6th

@melvin-bot melvin-bot bot removed the Overdue label Sep 4, 2024
Copy link

melvin-bot bot commented Sep 6, 2024

Payment Summary

Upwork Job

BugZero Checklist (@stephanieelliott)

  • I have verified the correct assignees and roles are listed above and updated the neccesary manual offers
  • I have verified that there are no duplicate or incorrect contracts on Upwork for this job (https://www.upwork.com/ab/applicants//hired)
  • I have paid out the Upwork contracts or cancelled the ones that are incorrect
  • I have verified the payment summary above is correct

@melvin-bot melvin-bot bot added the Overdue label Sep 7, 2024
@stephanieelliott
Copy link
Contributor

stephanieelliott commented Sep 7, 2024

Summarizing payment on this issue

Upwork job is here: https://www.upwork.com/jobs/~021832241147302028347

@melvin-bot melvin-bot bot added Daily KSv2 and removed Overdue Daily KSv2 labels Sep 7, 2024
@rayane-djouah
Copy link
Contributor

PR with no testing so the rate is $125

@stephanieelliott - The PR actually required testing, so I believe the reward should remain at the standard rate of $250.

I've accepted the job offer, thank you!

@melvin-bot melvin-bot bot added the Overdue label Sep 10, 2024
@stephanieelliott
Copy link
Contributor

Ah ok, thanks for the correction @rayane-djouah. I added an additional $125 milestone to the job, so the entire job should be $250. Both have been paid!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Daily KSv2 NewFeature Something to build that is a new item.
Projects
Status: Done
Development

No branches or pull requests

5 participants