project-lux / lux-middletier

Node backend for LUX frontend (a.k.a. middle tier)
Apache License 2.0
2 stars 0 forks source link

HAL Link for Sets on Event Pages #40

Closed roamye closed 2 months ago

roamye commented 7 months ago

Problem Description: the front end wishes to add missing data to set/collection pages by creating a HAL link that will add information of a Set to the Event.

Expected Behavior/Solution: Create a HAL link which returns the following query:

{"_scope": "event", used":{"id":"${SET_ID}"}} Hal LINK query name: lux:setEvents

Requirements: Once HAL link is created, Set information will appear under Event. (Please see mock of dependency issue)

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

UAT/LUX Examples: https://lux-front-tst.collections.yale.edu/view/set/664c4aeb-532f-46a2-9db8-4d0b51c73a64 should render information related to https://lux-front-tst.collections.yale.edu/view/activity/ef9bfa8b-1d19-4b44-a004-a5ab75cdf6d7

Dependencies/Blocks: Dependency: https://github.com/project-lux/lux-frontend/issues/88

Related Github Issues:

~Wireframe/Mockup:~ ~Place wireframe/mockup for the proposed solution at end of ticket.~

roamye commented 7 months ago

Re-created this issue as it was originally: https://github.com/project-lux/lux-middletier/issues/30 but it is hidden as ar2674 created that issue.

roamye commented 4 months ago

Should this be pushed to the next milestone instead of 05-28?

roamye commented 4 months ago

@clarkepeterf We need to know the name for the query HAL Link in order for this tix to move forward. Thank you.

roamye commented 4 months ago

@clarkepeterf in order for this ticket to move forward we need the name for the query HAL link. Please let me know what it is so I can update the tix. Thank you.

I have moved this tix to the next deployment cycle: 06/24 as it is still in prioritization review.

clarkepeterf commented 4 months ago

@roamye the HAL link name you created works for me

clarkepeterf commented 4 months ago

@clarkepeterf test this query with the example links

clarkepeterf commented 3 months ago

@kamerynB the bug is fixed, this query works now

kamerynB commented 3 months ago

@clarkepeterf Thank you Peter. I'll test it out

roamye commented 3 months ago

Approved by UAT - but this ticket will be kept open until the FE issue ( https://github.com/project-lux/lux-frontend/issues/88 ) has been approved/deployed.

roamye commented 3 months ago

Added to the release notes of 7/16 - but will close once FE 88 is approved/deployed in the next cycle.

roamye commented 3 months ago

Approved by UAT 7/22

roamye commented 2 months ago

FE88 looks good in prod. Closing.