MI-DPLA / combine

Combine /kämˌbīn/ - Metadata Aggregator Platform
MIT License
26 stars 11 forks source link

stateio: show only truly imported objects in "imports" list #344

Closed ghukill closed 5 years ago

ghukill commented 5 years ago

Currently, all imports that are mapped are written to imports part of import_manifest, when in fact some of them are not actually created.

For example, this Transformation that already exists:

...
Transformations: [
    {
        id: 1,
        name: "MODS to Service Hub profile"
    }
],
...

We cannot tell from the import_manifest that lacks the pk_hash if this is new or not, but it can be removed entirely from the imports section if the pk_hash shows the id does not change.

This would allow a "rollback" function, that would delete all model instances in the imports area, without risk of removing something that is in fact not new.

ghukill commented 5 years ago

done.