Skip to content

Simplify management of local state in import_groups app

Illya Klymov requested to merge xanf-simplify-import-entities-graphql into master

What does this MR do?

This MR removes previous bloated way of managing local state in import_groups app (based on heavy use of readFragment / writeFragment) with mutations which simply return relevant data

In other words we simply allow Apollo to do the job for us, instead of managing cache layer itself

So we:

  • add typedefs for easier understanding local types
  • split status field to separate type on progress which allows us to update progress both by group id (overwriting progress field) and by progress id (by simply returning relevant progress id as mutation response)

Screenshots (strongly suggested)

N/A. No visual changes :)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Discovered while implementing #300098 (closed)

Edited by Illya Klymov

Merge request reports