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

Updated godoc.org link to v2 docs. #4518

Merged
merged 1 commit into from
Oct 28, 2015
Merged

Conversation

lanzafame
Copy link

No description provided.

@otoolep
Copy link
Contributor

otoolep commented Oct 20, 2015

Thanks @lanzafame. Can you explain why we would make this change?

@lanzafame
Copy link
Author

Because the install instructions and examples on the client page references v2 but it was pointing at v1 on godoc.

@otoolep
Copy link
Contributor

otoolep commented Oct 21, 2015

OK, thaks @lanzafame -- have you signed the CLA?

@lanzafame
Copy link
Author

No. I will do once I'm at work. ~40 mins

@lanzafame
Copy link
Author

Where abouts is the CLA? I can't seem to find a link.

@beckettsean
Copy link
Contributor

It's hidden in the left nav bar of the docs, at the bottom.

https://influxdb.com/community/cla.html

@lanzafame
Copy link
Author

Cheers 👍

On Thu, 22 Oct 2015 at 09:46 Sean Beckett notifications@github.com wrote:

It's hidden in the left nav bar of the docs, at the bottom.

https://influxdb.com/community/cla.html


Reply to this email directly or view it on GitHub
#4518 (comment).

@lanzafame
Copy link
Author

And signed.

On Thu, 22 Oct 2015 at 10:02 Adrian Lanzafame adrianlanzafame92@gmail.com
wrote:

Cheers 👍

On Thu, 22 Oct 2015 at 09:46 Sean Beckett notifications@github.com
wrote:

It's hidden in the left nav bar of the docs, at the bottom.

https://influxdb.com/community/cla.html


Reply to this email directly or view it on GitHub
#4518 (comment).

@otoolep
Copy link
Contributor

otoolep commented Oct 22, 2015

CLA confirmed signed.

@sparrc

otoolep added a commit that referenced this pull request Oct 28, 2015
Updated godoc.org link to v2 docs.
@otoolep otoolep merged commit 6311a2b into influxdata:master Oct 28, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants