Skip to content

Conversation

@tcnichol
Copy link
Contributor

@tcnichol tcnichol commented Mar 1, 2023

This should now be fixed for all types. I created metadata definitions that are text input and date, and added the Unit metadata definition to test a dropdown.

Part of the problem was that 'contents' was used in place of 'content' in a few places so now it's always content for all metadata type. Also fieldName is an object, so using fieldName.name for the key for content[fieldName.name] to get actual metadata value. Metadata can now be created, value shown is right, and the updates work.

tcnichol added 3 commits March 1, 2023 14:15
problem was contents was used instead of content
fieldName was also object and not key, so using fieldName.name in place to get actual metadata value
@tcnichol tcnichol requested a review from longshuicy as a code owner March 1, 2023 20:34
@tcnichol tcnichol linked an issue Mar 1, 2023 that may be closed by this pull request
@tcnichol tcnichol requested review from ddey2 and lmarini March 1, 2023 20:35
@tcnichol tcnichol self-assigned this Mar 1, 2023
@tcnichol tcnichol added the bug Something isn't working label Mar 1, 2023
@tcnichol tcnichol added this to the Sprint Mar 2 2023 milestone Mar 1, 2023
Copy link
Member

@ddey2 ddey2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried and got Bad request error. No more info on backend. I deleted all the volumes and restarted docker containers, so it shouldn't have nay issue with db as well.

Screenshot 2023-03-02 at 8 50 58 AM

@tcnichol tcnichol requested a review from max-zilla as a code owner March 2, 2023 22:23
@tcnichol
Copy link
Contributor Author

tcnichol commented Mar 3, 2023

The error should be fixed now.

@max-zilla max-zilla requested a review from ddey2 March 6, 2023 15:11
@max-zilla max-zilla merged commit 37cb131 into main Mar 6, 2023
@max-zilla max-zilla deleted the 348-metadata-value-not-visible-on-ui branch March 6, 2023 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Metadata value not visible on UI

5 participants