biothings / discovery-app

The Data Discovery Engine project by the CD2H Data working-group
http://discovery.biothings.io
Apache License 2.0
4 stars 12 forks source link

Feature request: Allow addition of properties with bulk registration import #128

Closed flaneuse closed 2 years ago

flaneuse commented 2 years ago

A bulk registration .csv fails when only containing name/description properties for NIAID Data schema. The edit field only allows you to edit existing fields within the file, but you can never move forward since the required fields are missing.

It'd be nicer to be able to supplement a bulk registered dataset w/ handwritten properties for those missing fields (e.g. populate name, description and then allow the user to add the other fields via the form. Screen Shot 2022-02-15 at 1 09 17 PM

FWIW, I know this is a first pass of the tool, but I found the bulk registration box combined with the form to be really clunky-- I kinda ignored the "bulk registration" box and was confused why the form didn't have the name/description to be populated. Maybe more useful to convert the bulk registration box to something like a stepper, where you have one step per dataset which lists the overall stats (2/5 required fields, 1/10 recommended fields), and can move between the different datasets to register?

Also maybe useful: bulk add a missing property? (Author for all 7 datasets = Marco Cano, etc.)

flaneuse commented 2 years ago

Would also be good to alert the user what metadata you're throwing away from the import (fields that don't exist in the schema).

FWIW, this issue also applies to bulk .json import