USEPA / EPA_Non-geo_Metadata_Editor

3 stars 0 forks source link

Load metadata: populate distribution element #69

Closed aergul closed 5 years ago

aergul commented 5 years ago

It's a big section of metadata so it gets its own ticket

jzichichi commented 5 years ago

Loading of distribution content did not show up with this file:

7afb93f3-70e8-46c6-b8e7-a9bda388a12f.json.txt

aergul commented 5 years ago

This is because #65 is not implemented yet. Can you please try with a complete metadata record?

aergul commented 5 years ago

65 is now implemented and I was able to load above record.

jzichichi commented 5 years ago

better! I noticed something that seemed unusual tho. I load the file, it loads to the distribution elements, but if I try to then edit those loaded entries in that section, some seem to go missing. See the following showing this sequence:

load_distribution1 load_distribution2 load_distribution3

aergul commented 5 years ago

Good catch, fixed.

jzichichi commented 5 years ago

I'm still having a problem with the distribution section and retaining information. Media Type and Described by Type both get deleted when I click done editing this entry. See screenshot. I am not sure if this should be a separate ticket at this point, this was just from editing something to save it to then load it, but I never made it to the save part because I noticed this error.

untitled

aergul commented 5 years ago

I apologize, I think I forgot to deploy. Please try again.

jzichichi commented 5 years ago

Better tho format is getting lost now. Take a look. untitled

aergul commented 5 years ago

Format fixed

jzichichi commented 5 years ago

@aergul - looks great! I will pass to @torrin47 for review.

torrin47 commented 5 years ago

👍