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

Allow a name to be defined to be display in search results. #38

Merged
merged 2 commits into from
Oct 22, 2013
Merged

Allow a name to be defined to be display in search results. #38

merged 2 commits into from
Oct 22, 2013

Conversation

markprovan
Copy link
Contributor

While the current implementation uses the class name in the search results. It would be good to be able to define our own custom names. This PR addresses that by allowing the user to define this in the model:

def friendly_search_name
  "Document"
end

@gwagener
Copy link

Seems like a reasonable API. Since it's already using a helper there it seems like it would be better to add this logic to the helper.

@markprovan
Copy link
Contributor Author

@gwagener Agreed. Will make the relavent changes.

parndt added a commit that referenced this pull request Oct 22, 2013
Allow a name to be defined to be display in search results.
@parndt parndt merged commit d1d615d into refinery:master Oct 22, 2013
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.

None yet

3 participants