collectionspace / collectionspace-mapper

MIT License
0 stars 0 forks source link

Deprecate `check_terms` option #134

Closed kspurgin closed 2 years ago

kspurgin commented 2 years ago

It is not safe to ever use this option.

I am unable to figure out how to make the mapper generate exactly the same shortIDs and refnames as the UI/API would, so if you load data using mapper-generated terms that do not yet exist, and later load the same terms as authorities or create them as vocabularies, you may get mismatches.

kspurgin commented 2 years ago

Note: we are keeping check_record_status because sometimes you do know that you are adding all new records.

In this case, you can set check_record_status to false, and the status of all records will be set to :new for use by the collectionspace-csv-importer