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-08-01] [Workspace Feeds] [External] Create new Card Settings page #44311

Closed
mountiny opened this issue Jun 24, 2024 · 25 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Daily KSv2

Comments

@mountiny
Copy link
Contributor

Part of the Workspace feeds project.

Implement the following part of the design doc.

@koko57 @VickyStash @allgandalf @DylanDylann

Copy link

melvin-bot bot commented Jun 28, 2024

@MariaHCD, @mountiny Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@mountiny mountiny added Weekly KSv2 and removed Daily KSv2 labels Jun 28, 2024
@melvin-bot melvin-bot bot removed the Overdue label Jun 28, 2024
@mountiny
Copy link
Contributor Author

Later phase of implementation

@koko57
Copy link
Contributor

koko57 commented Jul 3, 2024

I can start working on this issue

@trjExpensify
Copy link
Contributor

Assigned you!

@koko57
Copy link
Contributor

koko57 commented Jul 4, 2024

WIP - changes are on this branch https://github.com/koko57/App/tree/feature/44311-card-settings-page. Tomorrow I'm ooo, I will continue on Monday.

@allgandalf
Copy link
Contributor

any update on this @koko57 ?

@koko57
Copy link
Contributor

koko57 commented Jul 9, 2024

@allgandalf working on it, but now prioritizing last tasks from phase 1

@koko57
Copy link
Contributor

koko57 commented Jul 11, 2024

Last 2 days I was mainly focused on #44305 and taken over the PR for #44315. Tomorrow I'm ooo, I will get back to it next week.

@koko57
Copy link
Contributor

koko57 commented Jul 15, 2024

Drafted PR for now - I need to rethink one thing and probably work on a solution with Maria and I also need to test it out on all the platforms.

@allgandalf
Copy link
Contributor

Thanks for the draft PR @koko57 , lets try to open up for review this week

@koko57
Copy link
Contributor

koko57 commented Jul 16, 2024

@allgandalf will be open today 😃

@koko57
Copy link
Contributor

koko57 commented Jul 16, 2024

@mountiny do we already have a page that "Learn more" should link to?

@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Weekly KSv2 labels Jul 16, 2024
@koko57
Copy link
Contributor

koko57 commented Jul 16, 2024

@allgandalf PR opened: #45415

@trjExpensify
Copy link
Contributor

I don't think it squarely fits, but the monthly settlement requirements are on this page: https://help.expensify.com/articles/expensify-classic/expensify-card/Admin-Card-Settings-and-Features

That said:

  1. It's an article in the Expensify Classic collection
  2. It talks about domains etc

So I think ideally, we need to link to a New Expensify article, which is tailored to workspace feeds. @joekaufmanexpensify is leading that here (internal link): https://github.com/Expensify/Expensify/issues/408330

@joekaufmanexpensify
Copy link
Contributor

Yep, the new help article I'm writing will include a section about the card settings page.

Copy link

melvin-bot bot commented Jul 23, 2024

⚠️ Looks like this issue was linked to a Deploy Blocker here

If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.

If a regression has occurred and you are the assigned CM follow the instructions here.

If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.

@allgandalf
Copy link
Contributor

allgandalf commented Jul 24, 2024

The linked blocker was actually know before deploy (comment here), we are still developing the links so we can track #46083 and address it once we have the links

Copy link

melvin-bot bot commented Jul 24, 2024

⚠️ Looks like this issue was linked to a Deploy Blocker here

If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.

If a regression has occurred and you are the assigned CM follow the instructions here.

If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.

@melvin-bot melvin-bot bot added Weekly KSv2 Awaiting Payment Auto-added when associated PR is deployed to production and removed Weekly KSv2 labels Jul 25, 2024
@melvin-bot melvin-bot bot changed the title [Workspace Feeds] [External] Create new Card Settings page [HOLD for payment 2024-08-01] [Workspace Feeds] [External] Create new Card Settings page Jul 25, 2024
@melvin-bot melvin-bot bot removed the Reviewing Has a PR in review label Jul 25, 2024
Copy link

melvin-bot bot commented Jul 25, 2024

Reviewing label has been removed, please complete the "BugZero Checklist".

Copy link

melvin-bot bot commented Jul 25, 2024

The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.11-5 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 2024-08-01. 🎊

For reference, here are some details about the assignees on this issue:

  • @koko57 does not require payment (Contractor)
  • @allgandalf requires payment (Needs manual offer from BZ)

@allgandalf
Copy link
Contributor

No payment needed here, this is part of a project

@melvin-bot melvin-bot bot added Daily KSv2 Overdue and removed Weekly KSv2 labels Jul 31, 2024
@allgandalf
Copy link
Contributor

not overdue

@melvin-bot melvin-bot bot removed the Overdue label Aug 1, 2024
Copy link

melvin-bot bot commented Aug 1, 2024

Issue is ready for payment but no BZ is assigned. @sonialiap you are the lucky winner! Please verify the payment summary looks correct and complete the checklist. Thanks!

Copy link

melvin-bot bot commented Aug 1, 2024

Payment Summary

Upwork Job

  • Contributor: @koko57 is from an agency-contributor and not due payment
  • ROLE: @allgandalf paid $(AMOUNT) via Upwork (LINK)

BugZero Checklist (@sonialiap)

  • 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

@sonialiap
Copy link
Contributor

Since no payment is needed, closing

@github-project-automation github-project-automation bot moved this from Release 2: Summer 2024 (Aug) to Done in [#whatsnext] #wave-collect Aug 2, 2024
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
Projects
Status: Done
Development

No branches or pull requests

7 participants