Abdelhafidh Belalia comments

Results 422 comments of


                                            Abdelhafidh Belalia

feature: implement qbo advanced configuration screens

## Reviewer Checklist - [X] I have verified the author checklist is complete (all boxes are checked off). - [X] I verified the correct issue is linked in the `###...

[$250] Request money - When trying to request money from the IOU report page, the "To" field is missing and the request button is greyed out

Was not able to reproduce either

[$250] Request money - When trying to request money from the IOU report page, the "To" field is missing and the request button is greyed out

This need reproduction

Fix cannot create task for new user via [] method

## Reviewer Checklist - [X] I have verified the author checklist is complete (all boxes are checked off). - [X] I verified the correct issue is linked in the `###...

[$250] Web - Task - Contact number is not set as an assignee followed by @expensify.sms via [] method

@nkdengineer Thanks for the proposal. Your RCA is correct. The solution looks good to me. 🎀 👀 🎀 C+ reviewed [Link to proposal](https://github.com/Expensify/App/issues/40518#issuecomment-2065884551)

[$250] Web - Task - Contact number is not set as an assignee followed by @expensify.sms via [] method

PR was deployed to production 2 weeks ago

[$250] Web - Task - Contact number is not set as an assignee followed by @expensify.sms via [] method

@johncschuster I don't think this needs a regression test. It's unlikely for this to resurface again.

[$500] Workspace - Invite members to workspace error but user can still join the workspace

@WaqasIbrahim Thanks for the proposal. Indeed the other user seem to "partly" get access to the workspace. I don't see them in the members list but according to the OP...

[$500] Workspace - Invite members to workspace error but user can still join the workspace

@laurenreidexpensify This seems internal. Can you please apple the label?

HIGH: [API Reliability] [$500] Multiple calls to `GetNewerActions` on report open

@shahinyan11 Thanks for the proposal. Your RCA is not complete. Can you explain why is the `loadNewerChats` function being called in the first place for the first report (report A)?...