Marco Chávez

Results 80 comments of Marco Chávez

I noticed when a report is submitted via scheduled submit, we're not sending the notification [here](https://github.com/Expensify/Web-Expensify/blob/577951eeaa07bc0b538b9f8567f7e41789d6d277/lib/ReportAPI.php#L3470-L3476) because of insufficient permission when calling `GetAccountIDsByEmails` using the [harverster authToken](https://github.com/Expensify/Web-Expensify/blob/fa8127b9ebfcf4be39b8e2b3b5b715cc62592f9b/script/bwm/ReportHarvester.php#L42).

Created a [PR](https://github.com/Expensify/Auth/pull/9914) to fix the harvester authToken issue. Seems the bugbot issue is not related to the schedule submit.

Created another [PR](https://github.com/Expensify/Auth/pull/9930) to log more data for the SubmitReport bugbot, which could also be related.

Probably also related to this [one](https://github.com/Expensify/App/issues/36541)

Fixing SSDos locally https://expensify.slack.com/archives/C03TQ48KC/p1708384890886629 to try to reproduce it

I submitted a request via SSDos locally and got the correct preview:

Waiting to be re-tested per `retest-weekly` label. Meanwhile I will move it to weekly cc @m-natarajan

Hi @mvtglobally just to be sure about the reproducible issue, in the video from the OP the report preview is updated with the Settlement button for the admin after they...

I followed the steps from the OP and I'm not really sure what's missing here. The report preview and LHN are updated: https://github.com/Expensify/App/assets/6829422/385a006d-15cd-4281-8db1-e0b58a5dff23 https://github.com/Expensify/App/assets/6829422/6e3032e5-fa1d-49ad-bbc5-ae02b32c6d4e From the expected behavior in the...

Discussed [here](https://expensify.slack.com/archives/C02NZ2HGJAZ/p1709236481460719?thread_ts=1709087964.400169&cid=C02NZ2HGJAZ) about closing it since instructions for expected output are not clear and the report preview issue hasn't happened lately. We can re-open or create another issue if another...