[Report Fields] [External] Disable changes to report fields when accounting integration is connected

Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.

https://new.expensify.com

MIT License

3.43k stars 2.8k forks source link

[Report Fields] [External] Disable changes to report fields when accounting integration is connected #43683

Closed mountiny closed 2 months ago

mountiny commented 3 months ago

[Main tracking issue].(https://github.com/Expensify/Expensify/issues/365155)

Design doc link.

Implement the following section of the design doc: https://docs.google.com/document/d/10b7pTVvgpB6_O8sRTKaCzZEVM3Ba6hxnIwyyQSSXCLY/edit#bookmark=id.dbhzh2rxut3y

This might be intertwined with the other sections too

Issue Owner

Current Issue Owner: @

melvin-bot[bot] commented 3 months ago

Current assignees @rushatgabhane and @shubham1206agra are eligible for the External assigner, not assigning anyone new.

melvin-bot[bot] commented 3 months ago

Triggered auto assignment to @anmurali (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details. Please add this Feature request to a GH project, as outlined in the SO.

waterim commented 3 months ago

Hello, Im Artem from Callstack and would like to help with this issue

rezkiy37 commented 3 months ago

Hi, I’m Michael (Mykhailo) from Callstack and I would like to work on this issue.

mountiny commented 3 months ago

@rezkiy37 will work on this one this week

mountiny commented 3 months ago

WIP PR is raised

rezkiy37 commented 3 months ago

I am working on the issue, but it depends on other issues (https://github.com/Expensify/App/issues/43685, https://github.com/Expensify/App/issues/43687).

rezkiy37 commented 2 months ago

Working on the issue.

rezkiy37 commented 2 months ago

I've opened the PR (https://github.com/Expensify/App/pull/44672) for review 🙂

rezkiy37 commented 2 months ago

Hello! FYI: I have a vacation from 11.07 to 26.07. Meanwhile, someone from Callstack can help with this issue.

mountiny commented 2 months ago

I believe this is done now too