peerkar / liferay-gsearch

Google like search for Liferay 7 CE and DXP
Other
56 stars 36 forks source link

Allow facet on knowledgebase articles #3

Closed denissignoretto closed 6 years ago

denissignoretto commented 6 years ago

If I'm not wrong knowledge base articles are not fully supported. I've tryed adding the km model to the gsearch configuration (and included related labels on resource bundle file in core-impl module) but I get no results searching by everything neighter facteing by kb articles. ... { "key": "knowledgebase", "entryClassName": "com.liferay.knowledgebase.model.KBArticle" }, ...

Thanks for any support to get it working with KB portlet too.

peerkar commented 6 years ago

Hi @denissignoretto. KB Portlet version 3.0.1 doesn't work in my (GSearch minimum requirement) DXP SP5 environment apparently because of newer kernel version. Which portal version are you running on? CE?

denissignoretto commented 6 years ago

Hi peerkar, I've tested version 1.0.0 of gsearch it on Liferay CE GA5 7.0.4.

And these are the knowledge base bundles I've installed:


g! lb -s knowledge
START LEVEL 20
   ID|State      |Level|Symbolic name
  514|Active     |   10|Liferay CE Knowledge Base (3.0.0)
  516|Active     |    1|com.liferay.knowledge.base.editor.configuration (1.0.1)
  517|Active     |   10|com.liferay.knowledge.base.api (4.5.0)
  518|Active     |   10|com.liferay.knowledge.base.item.selector.api (1.0.1)
  519|Active     |   10|com.liferay.knowledge.base.markdown.converter (1.0.6)
  520|Active     |   10|com.liferay.knowledge.base.service (1.1.14)
  521|Active     |   10|com.liferay.knowledge.base.item.selector.web (1.0.2)
  522|Active     |   10|com.liferay.knowledge.base.web (1.0.23)
peerkar commented 6 years ago

Hi @denissignoretto,

First, there's a typo in class name { "key": "knowledgebase", "entryClassName": "com.liferay.knowledgebase.model.KBArticle" },

That should be: { "key": "knowledgebase", "entryClassName": "com.liferay.knowledge.base.model.KBArticle" },

Secondly, I added the support for this class so you can just download new binaries (core-impl) and test.

denissignoretto commented 6 years ago

Hi @peerkar I've testd the new version and KB is now working! thank for supporting it!

peerkar commented 6 years ago

Glad it worked!