Skip to content

Conversation

@AntonEliatra
Copy link
Contributor

Description

Expanding conditional routing example

Version

all

Checklist

  • By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
    For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@github-actions
Copy link

Thank you for submitting your PR. The PR states are In progress (or Draft) -> Tech review -> Doc review -> Editorial review -> Merged.

Before you submit your PR for doc review, make sure the content is technically accurate. If you need help finding a tech reviewer, tag a maintainer.

When you're ready for doc review, tag the assignee of this PR. The doc reviewer may push edits to the PR directly or leave comments and editorial suggestions for you to address (let us know in a comment if you have a preference). The doc reviewer will arrange for an editorial review.

@kolchfa-aws
Copy link
Collaborator

@dlvenable Could you review this PR?

Signed-off-by: AntonEliatra <[email protected]>
@dlvenable
Copy link
Member

@san81 will take the technical review for this PR.

Copy link
Contributor

@san81 san81 left a comment

Choose a reason for hiding this comment

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

Technical review approved

Signed-off-by: AntonEliatra <[email protected]>
Copy link
Collaborator

@kolchfa-aws kolchfa-aws left a comment

Choose a reason for hiding this comment

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

Thank you, @AntonEliatra!

@kolchfa-aws kolchfa-aws added Editorial review PR: Editorial review in progress and removed Tech review PR: Tech review in progress labels Nov 6, 2025
@kolchfa-aws kolchfa-aws assigned natebower and unassigned dlvenable Nov 6, 2025
Copy link
Collaborator

@natebower natebower left a comment

Choose a reason for hiding this comment

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

LGTM

@natebower natebower removed the Editorial review PR: Editorial review in progress label Nov 6, 2025
@natebower natebower merged commit d138a91 into opensearch-project:main Nov 6, 2025
7 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Nov 6, 2025
* expanding on routes example

Signed-off-by: Anton Rubin <[email protected]>

* expanding on routes example

Signed-off-by: Anton Rubin <[email protected]>

* Update pipelines.md

Signed-off-by: AntonEliatra <[email protected]>

* Update pipelines.md

Signed-off-by: AntonEliatra <[email protected]>

* Apply suggestions from code review

Signed-off-by: kolchfa-aws <[email protected]>

---------

Signed-off-by: Anton Rubin <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>
Signed-off-by: kolchfa-aws <[email protected]>
Co-authored-by: kolchfa-aws <[email protected]>
(cherry picked from commit d138a91)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@AntonEliatra AntonEliatra deleted the expanding-conditional-routing-example branch November 7, 2025 10:40
aryasoni98 pushed a commit to aryasoni98/documentation-website that referenced this pull request Nov 11, 2025
* expanding on routes example

Signed-off-by: Anton Rubin <[email protected]>

* expanding on routes example

Signed-off-by: Anton Rubin <[email protected]>

* Update pipelines.md

Signed-off-by: AntonEliatra <[email protected]>

* Update pipelines.md

Signed-off-by: AntonEliatra <[email protected]>

* Apply suggestions from code review

Signed-off-by: kolchfa-aws <[email protected]>

---------

Signed-off-by: Anton Rubin <[email protected]>
Signed-off-by: AntonEliatra <[email protected]>
Signed-off-by: kolchfa-aws <[email protected]>
Co-authored-by: kolchfa-aws <[email protected]>
Signed-off-by: Arya Soni <[email protected]>
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.

5 participants