project-lux / lux-frontend

Web front end of LUX
Apache License 2.0
3 stars 0 forks source link

Add referred_to_by content for names #154

Open kamerynB opened 3 months ago

kamerynB commented 3 months ago

Problem Description: YPM will be vending referred_to_by data nested in identified_by data for names. The frontend does have a method of handling this at the moment.

Expected Behavior/Solution: Add support to the getNames() function for returning identified_by/referred_to_by/content.

Requirements:

Needed for promotion: If an item on the list is not needed, it should be crossed off but not removed.

UAT/LUX Examples:

Dependencies/Blocks:

Related Github Issues: N/A

Related links: N/A

~Wireframe/Mockup:~ TBD

kamerynB commented 1 month ago

@kkdavis14 Can you confirm if the data is ready? I can always add the functionality so that when the data is pushed the change is already in place.

prowns commented 1 month ago

@kamerynB and @kkdavis14 - are there UAT/LUX examples that could be added to this ticket?

kkdavis14 commented 1 month ago

it's not yet in the YPM data. Still blocked.

prowns commented 4 weeks ago

From UAT: need to update ticket to show wireframes and UAT examples. Use https://git.yale.edu/LUX/unit-data/issues/101 to define the front end impacts of this ticket. Not clear right now.

roamye commented 3 weeks ago

Currently the data for names() displayed on records come from the identified_by -> content portion of the JSON file (example with andy warhol names ).

Rather than get the data from this variable we will want to get it from the new referred_to_by variable. This ticket serves to add this new section nested under idenitified_by to get the name data from referred_to_by.

Since this is an addition to the json code, some UAT records may not be affected in the front end.

@kamerynB will reach out to @kkdavis14 to see if there is a UAT link we know for certain will be affected with this change. Otherwise we will have to wait till this is no longer blocked by: https://git.yale.edu/LUX/unit-data/issues/101