Skip to content

Conversation

johnstcn
Copy link
Member

@johnstcn johnstcn commented Feb 14, 2025

This PR provides a convenience function for parsing a map[string]string from a query parameter.

Context: #16558 (comment)

Copy link
Member

@mafredri mafredri left a comment

Choose a reason for hiding this comment

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

Nice addition! Could we also update existing code to use this implementation? Should serve to also verify its behavior.

@johnstcn johnstcn force-pushed the cj/queryparser-stringmap branch from 1ade009 to 4fec855 Compare February 18, 2025 12:56
@johnstcn
Copy link
Member Author

Nice addition! Could we also update existing code to use this implementation? Should serve to also verify its behavior.

Updated!

@johnstcn johnstcn merged commit 3a77375 into main Feb 18, 2025
30 checks passed
@johnstcn johnstcn deleted the cj/queryparser-stringmap branch February 18, 2025 14:14
@github-actions github-actions bot locked and limited conversation to collaborators Feb 18, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants