Skip to content

Conversation

@mouhc1ne
Copy link
Contributor

  • Add url_decode as a snapshot function.
  • Move common logic to both encode/decode to a parent class

@github-actions
Copy link
Contributor

github-actions bot commented Aug 28, 2025

🔍 Preview links for changed docs

@mouhc1ne mouhc1ne marked this pull request as ready for review August 28, 2025 20:52
@elasticsearchmachine elasticsearchmachine added the needs:triage Requires assignment of a team area label label Aug 28, 2025
@mouhc1ne mouhc1ne added >enhancement Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) :Analytics/ES|QL AKA ESQL ES|QL-ui Impacts ES|QL UI labels Aug 28, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

@elasticsearchmachine elasticsearchmachine removed the needs:triage Requires assignment of a team area label label Aug 28, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/kibana-esql (ES|QL-ui)

@mouhc1ne mouhc1ne requested review from ivancea and nik9000 August 28, 2025 22:02
Copy link
Contributor

@ivancea ivancea left a comment

Choose a reason for hiding this comment

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

Looks good!

@mouhc1ne mouhc1ne force-pushed the url_decode branch 2 times, most recently from 4b80e43 to c3b9ee5 Compare August 29, 2025 16:12
@mouhc1ne mouhc1ne requested a review from ivancea August 29, 2025 16:12
@mouhc1ne mouhc1ne self-assigned this Aug 29, 2025
@mouhc1ne mouhc1ne added the test-release Trigger CI checks against release build label Aug 29, 2025
@mouhc1ne mouhc1ne merged commit 88f55db into elastic:main Sep 2, 2025
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL >enhancement ES|QL-ui Impacts ES|QL UI Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) test-release Trigger CI checks against release build v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants