scientist-softserv / west-virginia-university

West Virginia University
0 stars 0 forks source link

🐛 Fix bookmark export #152

Closed kirkkwang closed 11 months ago

kirkkwang commented 11 months ago

Story

This commit will address a bug in the bookmark export. Previously, the export button on the bookmarks page would export with effectively no query params. This is because the params did not have a q and basically did a q="" search. If you had 1 bookmark then it'll just return the first result from an empty query search, which most likely would not be the bookmarked record you wanted.

This also enables the user to pick specific results via the bookmark feature and only export those results.

Expected Behavior Before Changes

Bookmark exports were not working as intended and only returned an empty query search hit.

Expected Behavior After Changes

Bookmark exports now are scoped to record identifiers and should return the expected hits.

Screenshots / Video

https://github.com/scientist-softserv/west-virginia-university/assets/19597776/d4763b45-1bbf-4488-b606-89104b56e360