Skip to content

Adds options.transformRequest to allow changing the structure of the request#772

Closed
Jendoliver wants to merge 1 commit into
devbridge:masterfrom
Jendoliver:master
Closed

Adds options.transformRequest to allow changing the structure of the request#772
Jendoliver wants to merge 1 commit into
devbridge:masterfrom
Jendoliver:master

Conversation

@Jendoliver
Copy link
Copy Markdown

(Defaults to previous implementation)

I just needed to do this to use an endpoint which was expecting the autocomplete search term in a nested object (something like the example I added to the docs). I agree that this is pretty strange but as modifying the endpoint is not an option and I think this could be the case for another person, my opinion is that this could be helpful.

I know there's a PR with a similar name (#179) but that one only changes the search term instead of changing the request structure.

If there's already a way to do this then sorry for not being able to find it!

defaulting to the previous implementation.
Updates docs.
@tkirda
Copy link
Copy Markdown
Member

tkirda commented May 20, 2026

Thanks for this contribution, and apologies for the very long silence.

The project was dormant for several years and has just shipped v2.0, which is a ground-up TypeScript rewrite with a new build (esbuild), test runner (Vitest + jsdom), and CI pipeline. As a result this PR no longer applies cleanly against master, and the surrounding code it touches has changed enough that mechanically rebasing it wouldn't be safe.

I'm closing it as part of a triage pass on the long-stale PR backlog — not because the idea lacks merit. If this change is still valuable to you against 2.0, please open a fresh PR (or an issue first if you'd like to confirm direction); I'll review new submissions against the current codebase promptly.

Thanks again for taking the time.

@tkirda tkirda closed this May 20, 2026
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.

2 participants