jeremytchang
jeremytchang
Hey @hirosassa again! Thanks for bringing this up. Let me look into this a little further and get back to you. My gut feeling is only `Content-Type` header is needed...
@jkaster What do you think of us deprecating `rtl.ApiSettings` for Go SDK? It deviates from our established ways to configure an SDK, either with environment variables or `looker.ini`.
@panoskoug There are 2 established ways for configuring the Go SDK for now. You can do it through [env variables](https://github.com/looker-open-source/sdk-codegen#environment-variable-configuration) or [looker.ini](https://github.com/looker-open-source/sdk-codegen#configuring-lookerini-or-env). Are you using `rtl.ApiSettings` in some way that...
Rebasing branch to see if it fixes the CI issue.
Hey @hirosassa! Our CI can only run on branches in our repo because of security reasons. I've copied your changes to this PR https://github.com/looker-open-source/sdk-codegen/pull/1193. When we squash merge the PR,...
For the future, we should have contributors point their PRs to main and merge their PRs into main. Or else this leaves the change owner unaware of their changes going...
Hey! This is a great call out. We already have generated [Error](https://github.com/looker-open-source/sdk-codegen/blob/main/go/sdk/v4/models.go#L1138) and [ValidationError](https://github.com/looker-open-source/sdk-codegen/blob/main/go/sdk/v4/models.go#L3617) struct types. We can make use of these types and json decode the error responses. To...
Thanks everyone for their input and discussion. It's great seeing active users of Looker's Go SDK! 🎉 And I'd love to learn more about your use cases! Currently, this thread...
As a follow up, a PR for custom headers in Go SDK. https://github.com/looker-open-source/sdk-codegen/pull/1288 @jessicasomaiya @JanReimerD
Fix here: https://github.com/looker-open-source/sdk-codegen/pull/1287