-
Notifications
You must be signed in to change notification settings - Fork 256
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
Enable the rendering pipeline to be configured per-instance #1996
Conversation
1 similar comment
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.
Approving despite a couple trivial suggestions.
@@ -2,27 +2,42 @@ | |||
|
|||
module Blacklight | |||
module Rendering | |||
# The field rendering pipeline | |||
# The field rendering pipeline. | |||
# This takes a field and it's values and transforms them through a list of |
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.
s/it's/its/
# @param values [Array] the values for the field | ||
# @param config [Blacklight::Configuration::Field] the field's configuration | ||
# @param document [SolrDocument] the document | ||
# @param context [Object] an execution context, used to execute the helper method in. |
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.
Trivial: two spaces between the
and helper
ceb8d6d
to
7fe9222
Compare
Thanks for the keen eye @mjgiarlo! |
@jcoyne Build went red. Restarted that build. Feel free to merge when this is green. ✔️ |
Rather than mutating the options passed to the pipeline.
7fe9222
to
285b7a6
Compare
No description provided.