Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Urn on solr doc #26

Merged
merged 5 commits into from
Aug 18, 2022
Merged

Urn on solr doc #26

merged 5 commits into from
Aug 18, 2022

Conversation

sephirothkod
Copy link
Contributor

No description provided.

id_arr << urn.gsub('.', '').gsub(':', '') # urn with punc stripped
if full_url = @item_solr['full_image_url_ssm'].presence
id_arr << urn = full_url&.split('/')&.last&.split('?')&.first&.gsub(':VIEW', '') # urn
id_arr << urn = urn&.upcase # .gsub('.', '').gsub(':', '') # urn with punc stripped
Copy link
Contributor

Choose a reason for hiding this comment

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

Are the gsubs in this line supposed to be commented out?

@bkiahstroud
Copy link
Contributor

I recommend merging #22 and #27 before merging this PR. This PR conflicts with both of them. It will be easier to resolve the conflicts after those are in main

@dl-maura dl-maura self-requested a review August 18, 2022 22:16
Copy link
Contributor

@dl-maura dl-maura left a comment

Choose a reason for hiding this comment

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

Tested on local

@dl-maura dl-maura merged commit a773230 into main Aug 18, 2022
@dl-maura dl-maura deleted the urn_on_solr_doc branch August 18, 2022 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants