-
Notifications
You must be signed in to change notification settings - Fork 5
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
Show page links directly to aeon, rather than a redirect #3559
Conversation
Also, refactor aeon url generation into its own class
dcec0f9
to
a78ac70
Compare
…fic holding or item
7c851b0
to
96c59e2
Compare
2ae7fcb
to
83198ff
Compare
83198ff
to
0277d63
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will make a separate ticket to deal with my comment on the volume.
expect(subject).to include('ItemNumber=32101071302192') | ||
end | ||
it('takes enumeration from the item') do | ||
expect(subject).to include('rft.volume=Vol+1%3A+no.+1+-+4') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Per Aeon's OpenURL mapping table it should accept "rft.volume" as a valid param for volume info, however by manual test it does not. It did accept just plain "volume". The Aeon form param is literally "ItemVolume", maybe for now we should pass enumeration as that param.
Thank you for adding the component and making the new Aeon OpenURL class @sandbergja. |
This passes information from both the host and constituent record, closing #3273
Still to-do: