search

uoregon-libraries / uolib-primo-nui

University of Oregon Libraries implementation of the Primo New UI
0 stars 0 forks source link

VE: Fix "report a problem" generated link #48

Closed jechols closed 2 years ago

jechols commented 2 years ago

This piece seems to work (I think?) but it's sending the wrong URL for VE. That needs to be changed to the VE URL.

e.g., if you visit https://alliance-uoregon.primo.exlibrisgroup.com/discovery/fulldisplay?docid=alma99122099390001451&context=L&vid=01ALLIANCE_UO:UO and activate "Report a Problem", you get sent to the "problem" page, but the URL there is reported as http://alliance-primo.hosted.exlibrisgroup.com/primo-explore/fulldisplay?vid=01ALLIANCE_UO:UO&docid=alma99122099390001451&context=L - this is obviously incorrect.

jechols commented 2 years ago

This is as fixed on the primo side as it can be. When we launch, we have to change how the library handles problem links, because the library code fills in the server (e.g., "https://alliance-uoregon.primo.exlibrisgroup.com"). So while the path is now fixed, the library site still have the old server info, and we can't safely change that until launch.

MargaretA-UO commented 2 years ago

And that’s something you will need to do, is that correct? I’ll add it to my go-live list.

From: Jeremy Echols @.> Sent: Monday, February 21, 2022 11:01 AM To: uoregon-libraries/uolib-primo-nui @.> Cc: Subscribed @.***> Subject: Re: [uoregon-libraries/uolib-primo-nui] VE: Fix "report a problem" generated link (Issue #48)

This is as fixed on the primo side as it can be. When we launch, we have to change how the library handles problem links, because the library code fills in the server (e.g., "https://alliance-uoregon.primo.exlibrisgroup.comhttps://urldefense.com/v3/__https:/alliance-uoregon.primo.exlibrisgroup.com__;!!C5qS4YX3!QRcGsat0m8lkiKZF49XrR5lpDmYlixWbRpZPItMvS2zMN5YgbgKJftcNgxggY-Z0uOI$"). So while the path is now fixed, the library site still have the old server info, and we can't safely change that until launch.

— Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/uoregon-libraries/uolib-primo-nui/issues/48*issuecomment-1047156282__;Iw!!C5qS4YX3!QRcGsat0m8lkiKZF49XrR5lpDmYlixWbRpZPItMvS2zMN5YgbgKJftcNgxggUHBC_XM$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/ASE4BQEWLVX2S62OF7XA5T3U4KDV3ANCNFSM5N64DCRA__;!!C5qS4YX3!QRcGsat0m8lkiKZF49XrR5lpDmYlixWbRpZPItMvS2zMN5YgbgKJftcNgxggJC-QG9s$. Triage notifications on the go with GitHub Mobile for iOShttps://urldefense.com/v3/__https:/apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675__;!!C5qS4YX3!QRcGsat0m8lkiKZF49XrR5lpDmYlixWbRpZPItMvS2zMN5YgbgKJftcNgxggsbBhAMU$ or Androidhttps://urldefense.com/v3/__https:/play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign*3Dnotification-email*26utm_medium*3Demail*26utm_source*3Dgithub__;JSUlJSU!!C5qS4YX3!QRcGsat0m8lkiKZF49XrR5lpDmYlixWbRpZPItMvS2zMN5YgbgKJftcNgxggsUVadwo$. You are receiving this because you are subscribed to this thread.Message ID: @.**@.>>

jechols commented 2 years ago

Yup, it's something I'll have to fix on the UO library site, which can't be set up prior to VE being ready (otherwise "problem" URLs will be wrong), and we can't even test the thing until after that fix happens, which means we can't test until after VE is live. Not ideal, but this isn't something we can avoid.