search

NASA-IMPACT / admg-backend

Apache License 2.0
2 stars 0 forks source link

Bugfix/445 cant publish gcmd instruments #453

Closed alukach closed 1 year ago

alukach commented 1 year ago

445 describes being unable to publish GCMD Instruments because the GCMD Instrument form renders without the ability to populate the gcmd_uuid data. This is due to the fact that we are not rendering UUID fields in our forms. I am a bit split on this issue. It seems that the only UUIDs that render on forms are the non-PK fields, so I don't believe it's actually bad to allow a user to submit the UUIDs. However, I am unsure that we actually would ever want to allow a user to manually create GCMD Instruments in the first place, isn't that something that the GCMD sync functionality should do for us?

Looking around the MI, the only record I could find that was affected by this change was the Alias model form, wherein we are now rendering the UUID of the record that the Alias relates to.

image
alukach commented 1 year ago

Closing this. We don't actually want users manually publishing GCMD items.