Skip to content
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

Save on edits to multiple tables #1569

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Save on edits to multiple tables #1569

wants to merge 4 commits into from

Conversation

JonasTreD
Copy link
Contributor

  • cleanup CaseStore and CaseView
  • add EditQueueContext to keep track of editQueue
  • Timeout moved to EditQueueContext, to prevent race conditions
  • GridRefContext added to store grid refs for campaigns, and grant campaigns access to alignedGridsRef
  • alignedGridRef passed from all tables, used to check if edits are ongoing in other grids
  • Campaign updates now grouped with resource Id, to allow multiple table updates for resources sharing the same resourceName
  • Components in CaseView added to EditQueueProvider where caseTabTable is used
  • Campaigns in CaseDrilling ScheduleTab added to GridRefProvider to share gridRefs

* cleanup CaseStore and CaseView
* add EditQueueContext to keep track of editqueue
* add componenets in CaseView to EditQueueProvider (work in progress, probably better to move this further down)
* Timeout moved to EditQueueContext, to prevent race conditions
* GridRefContext added to store grid refs for campaigns, and grant campaigns access to alignedGridsRef
* alignedGridRef passed from all tables, used to check if edits are ongoing in other grids
* Campaign updates now grouped with resource Id, to allow multiple table updates for resources sharing the same resourceName
* Components in CaseView added to EditQueueProvider where caseTabTable is used
* Campaigns in CaseDrilling ScheduleTab added to GridRefProvider to share gridRefs
@JonasTreD JonasTreD requested a review from arian-garshi March 7, 2025 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant