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

Update supervisord.conf #182

Closed
wants to merge 1 commit into from
Closed

Conversation

80982proton
Copy link

Add missing pimcore_search_backend_message reciver otherwise searching for relations does not work.

Add missing pimcore_search_backend_message reciver otherwise searching for relations does not work.
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@mattamon
Copy link
Contributor

@80982proton could you please sign the CLA

@jdreesen
Copy link
Contributor

When this gets merged, we probably don't need the info box here anymore: https://pimcore.com/docs/platform/Pimcore/Getting_Started/Installation/Docker_Based_Installation/#follow-these-steps

@fashxp
Copy link
Member

fashxp commented Mar 13, 2024

but when for what ever reason the simple search is not installed during installation, this transport won't be there and throwing errors, right?

@mattamon
Copy link
Contributor

@fashxp The Backend Search is neither enabled nor installed by default in the skeleton.
As you stated it would result in an error message. Just adding the messenger consume is not enough.

We could update the documentation that if you install skeleton, that you also need to enable and install the bundle.

@mattamon mattamon self-assigned this Mar 22, 2024
@mattamon
Copy link
Contributor

@80982proton I will close this PR now. See the following comment for details. #182 (comment)

For future PRs please also make sure to sign the CLA since we cannot accept PRs without it.

@mattamon mattamon closed this Mar 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants