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

Already on GitHub? Sign in to your account

Enhance Suffix#config so it can be usefully overridden by downstream #23

Merged
merged 1 commit into from Oct 8, 2013

Conversation

Projects
None yet
2 participants
Owner

cbeer commented Oct 8, 2013

apps by:

  • pushing the list of available suffixes and their order into the config
  • push type validation into the type_suffix proc
  • expose the config in a class-level method

I think this makes it possible to do a reasonable Solrizer 2.x-compatible suffix config, for those of us unfortunate enough to be stuck with the previous schema for some time to come.

This should be a minor-release compatible change, only the private config API has changed.

Enhance Suffix#config so it can be usefully overridden by downstream
apps by:

- pushing the list of available suffixes and their order into the config
- push type validation into the type_suffix proc
- expose the config in a class-level method

jcoyne added a commit that referenced this pull request Oct 8, 2013

Merge pull request #23 from projecthydra/suffixes-for-everyone
Enhance Suffix#config so it can be usefully overridden by downstream

@jcoyne jcoyne merged commit 0d857e0 into master Oct 8, 2013

1 check passed

default The Travis CI build passed
Details

@jcoyne jcoyne deleted the suffixes-for-everyone branch Oct 8, 2013

@awead awead referenced this pull request May 22, 2014

Closed

Bump to 3.1.2? #27

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment