Skip to content

Add optional limit parameter to suggest queries (#182)#193

Merged
fsteeg merged 4 commits into
masterfrom
182-limitParameter
Jul 10, 2025
Merged

Add optional limit parameter to suggest queries (#182)#193
fsteeg merged 4 commits into
masterfrom
182-limitParameter

Conversation

@fsteeg

@fsteeg fsteeg commented Jun 12, 2025

Copy link
Copy Markdown
Member

To resolve the remaining part of #182.

@netlify

netlify Bot commented Jun 12, 2025

Copy link
Copy Markdown

Deploy Preview for reconciliation-api-specs ready!

Name Link
🔨 Latest commit 83bac12
🔍 Latest deploy log https://app.netlify.com/projects/reconciliation-api-specs/deploys/686f8345c089bc0008a792e2
😎 Deploy Preview https://deploy-preview-193--reconciliation-api-specs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@saumier

saumier commented Jun 12, 2025

Copy link
Copy Markdown
Contributor

I my opinion, cursor should be renamed offset as a more common and developer friendly term. This would work pair nicely with limit.

@wetneb wetneb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I agree that offset matches the description of the field better (but to me the terms aren't interchangeable).

@fsteeg fsteeg merged commit c9a20ba into master Jul 10, 2025
5 checks passed
@fsteeg fsteeg deleted the 182-limitParameter branch July 10, 2025 14:26
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.

3 participants