Skip to content
This repository has been archived by the owner on Apr 28, 2019. It is now read-only.

added searchable to content_fields #19

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

greeneca
Copy link
Contributor

Added searchable to content_file for the search gem

@wouterw
Copy link

wouterw commented Nov 26, 2013

Would be nice to get this one merged. 👍

@crburton
Copy link

@did any reason as to why we cannot have this merged - it is really annoying having this error 👍

@did
Copy link
Member

did commented Feb 26, 2014

@crburton, well, you're right. But, if we add it, people who don't install the search gem in their Engine installation will have issues with it when pushing their sites.
It's a real dilemma for me. Hopefully, that will be solved by the V3 but by then, I don't know which is the best solution. Any ideas?

@neuronaut
Copy link

If wagon won't let us upload content types with some fields marked searchable then how do we get those content types to be searchable? I'm just following the documentation found here: http://doc.locomotivecms.com/guides/search, and now I find you are intentionally not adding the fix that allows the method described in the docs to work. Could you please at least update your documentation with the correct instructions? Would be MUCH appreciated!

@ellimist
Copy link

Anyone found a way to solve this problem? It's been 10 months since this fix was posted, and still not merged into master :| Any other way I could add the search functionality to my wagon and still be able to push it to the engine?

Thank you.

@greeneca
Copy link
Contributor Author

Dider is not going to merge this in because not everyone uses the search
feature. If they didn't it would be broken for them. For now you will have
to pull this patch in manually.

Charlie Greene
[email protected]

On Mon, Aug 18, 2014 at 9:50 AM, Adrian Bordinc [email protected]
wrote:

Anyone found a way to solve this problem? It's been 10 months since this
fix was posted, and still not merged into master :| Any other way I could
add the search functionality to my wagon and still be able to push it to
the engine?

Thank you.


Reply to this email directly or view it on GitHub
#19 (comment).

@did
Copy link
Member

did commented Aug 18, 2014

thanks @greeneca, actually, if you link the "locomotivecms-search" gem to your Wagon site Gemfile, it might work. Please use the master branch of Wagon.

@ellimist
Copy link

I have the latest wagon and I have added the search gem both in the wagon site and the locomotive, but it still fails to push or serve.

From the logs:
"[Locomotive::Mounter::Models::ContentField] setting an unknown attribute 'searchable' with the value 'true'"

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants