Skip to content

Add ability to query backend with multiple IDs#741

Merged
garrettjstevens merged 3 commits intomainfrom
fetch-multiple-features
Dec 10, 2025
Merged

Add ability to query backend with multiple IDs#741
garrettjstevens merged 3 commits intomainfrom
fetch-multiple-features

Conversation

@shashankbrgowda
Copy link
Copy Markdown
Contributor

Fixes #736

Copy link
Copy Markdown
Contributor

@garrettjstevens garrettjstevens left a comment

Choose a reason for hiding this comment

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

Could you make this a Get request instead of a Post, like the example in #736?

@garrettjstevens
Copy link
Copy Markdown
Contributor

As discussed in meeting just now, this is Post because too many IDs can make the URL too long for a Get request.

@garrettjstevens garrettjstevens force-pushed the fetch-multiple-features branch from 75ccbe3 to 18b1d3a Compare December 9, 2025 18:25
@garrettjstevens
Copy link
Copy Markdown
Contributor

I changed the naming of the endpoints a bit. I want to keep the existing one as getFeatures even though getFeaturesByRange is a bit more specific because getFeatures is a sort of JBrowse idiom that I'd like to keep consistent with.

@garrettjstevens garrettjstevens force-pushed the fetch-multiple-features branch from 18b1d3a to 1933f48 Compare December 9, 2025 18:31
@garrettjstevens garrettjstevens changed the title Query backend with multiple IDs Add ability to query backend with multiple IDs Dec 10, 2025
@garrettjstevens garrettjstevens merged commit 8666975 into main Dec 10, 2025
7 checks passed
@garrettjstevens garrettjstevens deleted the fetch-multiple-features branch December 10, 2025 22:29
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.

Query backend with multiple IDs

2 participants