Skip to content

Conversation

@mnocon
Copy link
Contributor

@mnocon mnocon commented Jan 21, 2026

Documentation for Elasticsearch 8 support in DXP 4.6

@mnocon mnocon changed the base branch from 5.0 to 4.6 January 22, 2026 10:20
@mnocon mnocon changed the title es 8 4.6 [4.6] Doc for Elasticsearch 8 Jan 22, 2026
@mnocon mnocon marked this pull request as ready for review January 22, 2026 12:46
@mnocon mnocon requested review from barw4 and micszo January 22, 2026 14:03
Copy link
Contributor

@konradoboza konradoboza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very descriptive! One nitpick:

Co-authored-by: Konrad Oboza <konrad.oboza@ibexa.co>
@mnocon mnocon requested a review from a team January 27, 2026 12:06
@ibexa-workflow-automation-1 ibexa-workflow-automation-1 bot requested review from adriendupuis, dabrt and julitafalcondusza and removed request for a team January 27, 2026 12:07
Copy link
Contributor

@dabrt dabrt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please see my comments, some are preferential. If you agree with them, you may need to revisit the E8 doc for 5.x

For more information, see:

- Elasticsearch 7: [SSL Encryption](https://www.elastic.co/guide/en/elasticsearch/client/php-api/7.x/connceting.html#ssl-encryption)
- Elasticsearch 8: [SSL Encryption](https://www.elastic.co/guide/en/elasticsearch/client/php-api/8.19/connecting.html#ssl-encryption)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dead link

mnocon and others added 2 commits January 29, 2026 10:53
Co-authored-by: Tomasz Dąbrowski <64841871+dabrt@users.noreply.github.com>
@mnocon
Copy link
Contributor Author

mnocon commented Jan 29, 2026

Thank you Tomek, I've applied your suggestions in af55521 and d2c07e4

@mnocon mnocon requested a review from dabrt January 29, 2026 10:44
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.

3 participants