-
Notifications
You must be signed in to change notification settings - Fork 10
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
fix(javascript): remove factory from build #639
Conversation
✅ Deploy Preview for api-clients-automation ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✗ The generated branch has been deleted.If the PR has been merged, you can check the generated code on the |
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.
Looks good !
{{#operations}} | ||
import type { | ||
{{#operation}} | ||
{{#vendorExtensions.x-create-wrapping-object}} | ||
{{#lambda.titlecase}}{{nickname}}{{/lambda.titlecase}}Props, | ||
{{/vendorExtensions.x-create-wrapping-object}} | ||
{{/operation}} | ||
} from '../model/clientMethodProps'; | ||
{{/operations}} | ||
|
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.
import *
is not enough here ?
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.
If we give it an alias we would have to change all methods to use Alias.TypeName
so I thought it was less verbose to do it at the import level, wdyt?
🧭 What and Why
🎟 JIRA Ticket: https://algolia.atlassian.net/browse/APIC-506
Changes included:
We were exposing the factory function in the final build, which could be confusing for a user when importing the client using IDE autocompletion.
We now move the method props to a
clientMethodProps
model file.Side changes:
search
client foralgoliasearch
algoliasearch
client🧪 Test
CI :D