Skip to content

Conversation

ketkee-aryamane
Copy link
Contributor

@ketkee-aryamane ketkee-aryamane commented Sep 26, 2025

Linked to elastic/docs-content#2617
Cleanup the PHP client overview page
Add info for Serverless as mentioned in the Git page
add subs in docsets

add serverless info
add subs to docsets
Copy link

github-actions bot commented Sep 26, 2025

🔍 Preview links for changed docs

@ketkee-aryamane ketkee-aryamane changed the title cleanup the overview page sync the Git page and the docs page for PHP client overview Sep 26, 2025
@ketkee-aryamane ketkee-aryamane marked this pull request as ready for review September 26, 2025 11:04
@ketkee-aryamane ketkee-aryamane requested a review from a team as a code owner September 26, 2025 11:04
Copy link
Contributor

@ezimuel ezimuel left a comment

Choose a reason for hiding this comment

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

Thanks for this PR, it looks awesome. I left one small note that contains two useful external links (in elastic.co and search labs) that maybe can be added in the documentation.

Copy link
Contributor

@ezimuel ezimuel left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@marciw marciw left a comment

Choose a reason for hiding this comment

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

LGTM, thanks! A couple of suggestions but approving. 👍

@ketkee-aryamane ketkee-aryamane merged commit 4950e69 into elastic:main Sep 26, 2025
19 checks passed
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.

4 participants