-
Notifications
You must be signed in to change notification settings - Fork 21
feat(javascript): add accountCopyIndex
helper
#4798
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
💥 Breaking changes detected !!Either this PR or a previous PR not released yet introduced breaking changes, be careful when merging. ✔️ Code generated!
📊 Benchmark resultsBenchmarks performed on the method using a mock server, the results might not reflect the real-world performance.
|
92da4a7
to
dec43f7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good idea !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for the test !
destinationAppID: string; | ||
|
||
/** | ||
* The API Key of the `destinationAppID` to write the index to, must have write ACLs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe be more precise with an ACL list ? we have the Required ACL
section in most endpoints
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done!
filterPromotes: true, | ||
promote: [ | ||
{ | ||
objectIDs: ['\u00C6on Flux'], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❤️
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all good for me, gg !
…skip ci] Co-authored-by: Clément Vannicatte <[email protected]>
algolia/api-clients-automation#4798 Co-authored-by: algolia-bot <[email protected]> Co-authored-by: Clément Vannicatte <[email protected]>
🧭 What and Why
🎟 JIRA Ticket: https://algolia.atlassian.net/browse/DI-3725
Changes included:
This helper wasn't added to the generated clients as we did not had the other internal helpers that are used, but now that we have them, and customer requests this method to be added, it's pretty easy to do so!