-
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
[HOLD for payment 2024-03-06][$500] [Simplified Collect][Categories] WorkspaceCategoriesPage: base page implementation, displaying categories and its statuses #35705
Comments
Triggered auto assignment to @jliexpensify ( |
@ArekChr can you please comment here so I can assign you to the issue? |
Sure, please assign me @luacmartins |
@luacmartins, @ArekChr, @jliexpensify Whoops! This issue is 2 days overdue. Let's get this updated quick! |
Job added to Upwork: https://www.upwork.com/jobs/~014e6c1796715bbf98 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @parasharrajat ( |
The draft PR is nearly complete. @luacmartins @jliexpensify, could you please review my latest update in the PR? The issue mentioned in comment is currently a blocker, thanks! |
Pulled the Q out to Slack |
What's the status of this issue? Looks like it is done cc: @trjExpensify |
Yes, it's done. PR was deployed to production yesterday. I'll manually update the title. |
Yeah, we need to switch @parasharrajat for @rushatgabhane for the C+ tracking. |
@jliexpensify @luacmartins I have an automatic issue to issue payment for the PR that's linked to this issue. Do we need to issue one or two payments to Rushat for #36324? |
@sonialiap thanks for checking. We should only issue one payment for @rushatgabhane's PR review. I think we can close the internal issue in favor of this one. |
Thanks! Makes sense to keep it all in one place 👍 Closing out the other issue |
Payment Summary
If this is correct, I'll remove the Upworks job. @rushatgabhane anything need to be done for the checklist? |
$500 payment approved for @rushatgabhane based on summary. |
Upworks job removed, bump on @rushatgabhane for the checklist (if applicable). |
@jliexpensify it's a new feature. So I recommend that we add the test steps in regression suite.
|
Payment Summary
BugZero Checklist (@jliexpensify)
|
Payment Summary:
|
Implement this section of the Simplified Collect: Profile, Members, Categories design doc.
Issue Owner
Current Issue Owner: @parasharrajatUpwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: