-
Notifications
You must be signed in to change notification settings - Fork 3.2k
[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
Comments
Triggered auto assignment to @slafortune ( |
@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! |
@luacmartins #60913 will be fixed by @borys3kk in his PR |
@luacmartins I've opened PR for #59927 - it's here #60957 |
@luacmartins I think #59905 was fixed by some search related changes |
@luacmartins for #59876 I'm getting this error when trying to move IOU report to workspace: I think it's backend issue 🤔 |
Doesn't Carlos have a draftPR up for this one here: #60872? |
Yea, I'll address that one in my PR |
Sounds good! |
@luacmartins is this #61049 (comment) connected to this project? 😅 |
Hi |
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 |
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 |
|
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 / @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] |
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. |
Payment Summary
BugZero Checklist (@slafortune)
|
|
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 / @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] |
Let's use this issue to track any follow ups from #59999
Needs work
*HeaderOld.tsx
filesIn review
Fixed
Issue Owner
Current Issue Owner: @slafortuneThe text was updated successfully, but these errors were encountered: