Skip to content
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

Default Client #59

Closed
wants to merge 1 commit into from
Closed

Default Client #59

wants to merge 1 commit into from

Conversation

sethgrid
Copy link

@sethgrid sethgrid commented May 5, 2016

Previously, if you made a client:
sg := sendgrid.NewSendGridClientWithApiKey(sendgridKey)
Then you could not do sg.Client.Timeout = 30 * time.Second. You would get a nil reference.

Previously, if you made a client:
sg := sendgrid.NewSendGridClientWithApiKey(sendgridKey)
Then you could not do sg.Client.Timeout = 30 * time.Second. You would get a nil reference.
@thinkingserious
Copy link
Contributor

Thanks for the pull request @sethgrid,

This library has been rewritten and will be replaced in a few weeks with this one: https://github.com/sendgrid/sendgrid-go/tree/v3beta

Please submit all pull requests agains the v3beta branch.

Thanks!

With Best Regards,

Elmer

@thinkingserious thinkingserious added type: community enhancement feature request not on Twilio's roadmap status: help wanted requesting help from the community labels May 22, 2016
@broady
Copy link

broady commented Jul 19, 2016

-1 on this approach. just let people set their own.

@thinkingserious
Copy link
Contributor

Thanks for chiming in @broady, especially before we started work on this. Much appreciated!

@thinkingserious
Copy link
Contributor

#73

@childish-sambino childish-sambino deleted the missing-client branch January 28, 2020 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: help wanted requesting help from the community type: community enhancement feature request not on Twilio's roadmap
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants