Skip to content

[Due for payment 2025-05-22] [Due for payment 2025-05-14] [Simplified Actions] FOLLOWUP: Add Primary and Secondary actions to report previews and header #60852

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

Open
13 of 14 tasks
luacmartins opened this issue Apr 24, 2025 · 24 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Weekly KSv2

Comments

@luacmartins
Copy link
Contributor

luacmartins commented Apr 24, 2025

Let's use this issue to track any follow ups from #59999

Needs work

  • Remove beta flag and delete *HeaderOld.tsx files

In review

Fixed

Issue OwnerCurrent Issue Owner: @slafortune
@luacmartins luacmartins added Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 labels Apr 24, 2025
Copy link

melvin-bot bot commented Apr 24, 2025

Triggered auto assignment to @slafortune (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@luacmartins
Copy link
Contributor Author

luacmartins commented Apr 25, 2025

@jnowakow @borys3kk I retested all issues reported by QA and added the ones that are still reproducible to the OP. @borys3kk has a draft PR to address a few of them already. It seems like one of them is coming from the BE so I'll take a look at that one. @jnowakow could you pick one of the other issues that are not addressed yet and work on a PR for it please? Let's push to remove the beta next week!

@jnowakow
Copy link
Contributor

jnowakow commented Apr 28, 2025

@luacmartins #60913 will be fixed by @borys3kk in his PR

@trjExpensify trjExpensify moved this to Second Cohort - CRITICAL in [#whatsnext] #migrate Apr 28, 2025
@jnowakow
Copy link
Contributor

@luacmartins I've opened PR for #59927 - it's here #60957

@jnowakow
Copy link
Contributor

@luacmartins I think #59905 was fixed by some search related changes
Image

@jnowakow
Copy link
Contributor

@luacmartins for #59876 I'm getting this error when trying to move IOU report to workspace:
Image

I think it's backend issue 🤔

@trjExpensify
Copy link
Contributor

trjExpensify commented Apr 28, 2025

@luacmartins for #59876 I'm getting this error when trying to move IOU report to workspace:

Doesn't Carlos have a draftPR up for this one here: #60872?

@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Daily KSv2 labels Apr 29, 2025
@luacmartins
Copy link
Contributor Author

Yea, I'll address that one in my PR

@trjExpensify
Copy link
Contributor

Sounds good!

@jnowakow
Copy link
Contributor

@luacmartins is this #61049 (comment) connected to this project? 😅

@borys3kk
Copy link
Contributor

borys3kk commented Apr 30, 2025

Hi
Regarding the remaining issues:
$ #59876 is taken caren by you Carlos
$ #59905 was fixed as stated in this comment: $ #60852 (comment)
$ #59893 was fixed in my pr
$ #61092 is I think expected behaviour but waiting for response from tester if it's a bug I will take care of this
$ #61049 fixed by you Carlos
$ #61019 you're taking care of this on as well
so all in all I think all issues from this follow up are looked after :)

@luacmartins
Copy link
Contributor Author

@luacmartins is this #61049 (comment) connected to this project? 😅

Not really, but I think it's a cleanup item that we should address sooner rather than later. It's at the bottom of the list priority wise though

@luacmartins
Copy link
Contributor Author

#59905 was fixed as stated in this comment: $ #60852 (comment)

I can still reproduce this issue following the steps in the OP (except for the gmail part)

Screen.Recording.2025-04-30.at.3.59.51.PM.mov

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Weekly KSv2 labels May 5, 2025
@melvin-bot melvin-bot bot added Weekly KSv2 Awaiting Payment Auto-added when associated PR is deployed to production and removed Weekly KSv2 labels May 7, 2025
@melvin-bot melvin-bot bot changed the title [Simplified Actions] FOLLOWUP: Add Primary and Secondary actions to report previews and header [Due for payment 2025-05-14] [Simplified Actions] FOLLOWUP: Add Primary and Secondary actions to report previews and header May 7, 2025
Copy link

melvin-bot bot commented May 7, 2025

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

@melvin-bot melvin-bot bot removed the Reviewing Has a PR in review label May 7, 2025
Copy link

melvin-bot bot commented May 7, 2025

The solution for this issue has been 🚀 deployed to production 🚀 in version 9.1.40-7 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 2025-05-14. 🎊

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

  • @mananjadhav requires payment through NewDot Manual Requests
  • @suneox requires payment (Needs manual offer from BZ)
  • @borys3kk does not require payment (Contractor)
  • @jnowakow does not require payment (Contractor)

Copy link

melvin-bot bot commented May 7, 2025

@mananjadhav / @suneox @slafortune @mananjadhav / @suneox The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed. Please copy/paste the BugZero Checklist from here into a new comment on this GH and complete it. If you have the K2 extension, you can simply click: [this button]

@luacmartins
Copy link
Contributor Author

I think all the PRs fixing the bugs in the OP are merged. @jnowakow let's update #61378 to get it merged tomorrow!

@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 Daily KSv2 and removed Weekly KSv2 labels May 8, 2025
Copy link

melvin-bot bot commented May 13, 2025

⚠️ 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.

Copy link

melvin-bot bot commented May 14, 2025

Payment Summary

Upwork Job

  • Reviewer: @mananjadhav owed $250 via NewDot
  • ROLE: @suneox paid $(AMOUNT) via Upwork (LINK)
  • Contributor: @borys3kk is from an agency-contributor and not due payment
  • Contributor: @jnowakow is from an agency-contributor and not due payment

BugZero Checklist (@slafortune)

  • I have verified the correct assignees and roles are listed above and updated the necessary 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 Weekly KSv2 and removed Daily KSv2 Weekly KSv2 labels May 15, 2025
@melvin-bot melvin-bot bot changed the title [Due for payment 2025-05-14] [Simplified Actions] FOLLOWUP: Add Primary and Secondary actions to report previews and header [Due for payment 2025-05-22] [Due for payment 2025-05-14] [Simplified Actions] FOLLOWUP: Add Primary and Secondary actions to report previews and header May 15, 2025
@melvin-bot melvin-bot bot removed the Reviewing Has a PR in review label May 15, 2025
Copy link

melvin-bot bot commented May 15, 2025

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

Copy link

melvin-bot bot commented May 15, 2025

The solution for this issue has been 🚀 deployed to production 🚀 in version 9.1.45-21 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 2025-05-22. 🎊

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

  • @mananjadhav requires payment through NewDot Manual Requests
  • @suneox requires payment (Needs manual offer from BZ)
  • @borys3kk does not require payment (Contractor)
  • @jnowakow does not require payment (Contractor)

Copy link

melvin-bot bot commented May 15, 2025

@mananjadhav / @suneox @slafortune @mananjadhav / @suneox The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed. Please copy/paste the BugZero Checklist from here into a new comment on this GH and complete it. If you have the K2 extension, you can simply click: [this button]

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 Bug Something is broken. Auto assigns a BugZero manager. Weekly KSv2
Projects
Status: Second Cohort - CRITICAL
Development

No branches or pull requests

7 participants