Mohammad Luthfi Fathur Rahman
Mohammad Luthfi Fathur Rahman
### Proposal I believe this error log happen because the API `AuthenticatePusher` got called before API `AddWorkspaceRoom` when we navigated to the report screen. It might be making the report...
@dangrous Thanks for the feedback! Yeah I guess you're right about `Report.subscribeToReportTypingEvents(this.props.report.reportID);` is causing the issue. How about we call `Report.subscribeToReportTypingEvents(this.props.report.reportID);` after `this.props.report.pendingFields.addWorkspaceRoom` value is null? Since the optimistic data...
### Proposal Since the root cause is defined [here](https://github.com/Expensify/App/blob/88515e4d22b346a197f9a698a562f15e937f6be0/src/pages/home/report/ReportActionsView.js#L103) we need to delay call of `Report.subscribeToReportTypingEvents` after the API command `AddWorkspaceRoom` is done. **Solution** We can call `Report.subscribeToReportTypingEvents` after the...
I just try your @dangrous snippet code, but I still got the issue without throttling the network. `AuthenticatePusher` is still called before `AddWorkspaceRoom` and returns a `404` error. How about...
Sorry for the long wait @dangrous and thanks for the suggestions! ### Proposal Move the call of `Report.subscribeToReportTypingEvents(this.props.report.reportID)` to `componentDidUpdate`. We will wait for `pendingFields.addWorkspaceRoom` success which changes the value...
@thesahindia For the report that was already created doesn't have `pendingFields` or has `pendingFields` but not with the `addWorkspaceRoom`. So I guess we need that.
### Proposal This happens because when we paste with `cmd+v` or the clipboard type is `text/html` there's an empty image tag. While we have the empty tag ``, we try...
### Proposal **Root cause** We have a keyboard listener for the `Enter` key and it's also captured on input. https://github.com/Expensify/App/blob/d3120758546153d25002866f6cd770d6b2b123c2/src/components/OptionsSelector/BaseOptionsSelector.js#L53-L67 On Desktop/Web can use `CMD+Enter` to create a new line...
Bump @eVoloshchak 🚀
@eVoloshchak Yup I'll try it and merge the latest `main`.