Skip to content

Conversation

@shortcuts
Copy link
Member

@shortcuts shortcuts commented Apr 4, 2022

🧭 What and Why

🎟 JIRA Ticket: https://algolia.atlassian.net/browse/APIC-412

Changes included:

Follow up of #313

The method's query parameters accepted only strings, while we could allow an object to ease the usage. This PR updates the spec and templates to reflect this change

🧪 Test

CI :D

@shortcuts shortcuts self-assigned this Apr 4, 2022
@netlify
Copy link

netlify bot commented Apr 4, 2022

Deploy Preview for api-clients-automation canceled.

Name Link
🔨 Latest commit 24f96c2
🔍 Latest deploy log https://app.netlify.com/sites/api-clients-automation/deploys/624c1c2e4f25b800083e3055
@algolia-bot
Copy link
Collaborator

algolia-bot commented Apr 4, 2022

✗ The generated branch has been deleted.

If the PR has been merged, you can check the generated code on the generated/main branch.

@shortcuts shortcuts changed the title fix(templates): fix wrong query parameters fix(template): fix wrong query parameters Apr 4, 2022
@shortcuts shortcuts changed the title fix(template): fix wrong query parameters fix(spec): fix wrong query parameters Apr 4, 2022
@shortcuts shortcuts marked this pull request as ready for review April 5, 2022 09:02
@shortcuts shortcuts requested review from a team, damcou and eunjae-lee and removed request for a team April 5, 2022 09:02
@shortcuts shortcuts enabled auto-merge (squash) April 5, 2022 09:13
@shortcuts shortcuts disabled auto-merge April 5, 2022 09:28
damcou
damcou previously approved these changes Apr 5, 2022
Copy link
Contributor

@damcou damcou left a comment

Choose a reason for hiding this comment

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

Looks ok on the php side

Copy link
Collaborator

@millotp millotp left a comment

Choose a reason for hiding this comment

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

Awesome !

@shortcuts shortcuts enabled auto-merge (squash) April 5, 2022 10:39
@shortcuts shortcuts disabled auto-merge April 5, 2022 10:41
@shortcuts shortcuts enabled auto-merge (squash) April 5, 2022 10:41
@shortcuts shortcuts merged commit cf25c6a into main Apr 5, 2022
@shortcuts shortcuts deleted the fix/customRequest-queryParams branch April 5, 2022 10:48
algolia-bot added a commit to algolia/algoliasearch-client-javascript that referenced this pull request Apr 5, 2022
algolia/api-clients-automation#330 Co-authored-by: Clément Vannicatte <20689156+shortcuts@users.noreply.github.com>
algolia-bot added a commit to algolia/algoliasearch-client-java that referenced this pull request Apr 5, 2022
algolia/api-clients-automation#330 Co-authored-by: Clément Vannicatte <20689156+shortcuts@users.noreply.github.com>
algolia-bot added a commit to algolia/algoliasearch-client-php that referenced this pull request Apr 5, 2022
algolia/api-clients-automation#330 Co-authored-by: Clément Vannicatte <20689156+shortcuts@users.noreply.github.com>
shortcuts added a commit that referenced this pull request Apr 22, 2022
Co-authored-by: Pierre Millot <pierre.millot@algolia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants