Skip to content

fix default changes timeout value#1432

Merged
pasin merged 1 commit intocouchbase:masterfrom
swaheed2:patch-2
Sep 7, 2016
Merged

fix default changes timeout value#1432
pasin merged 1 commit intocouchbase:masterfrom
swaheed2:patch-2

Conversation

@swaheed2
Copy link
Copy Markdown
Contributor

@swaheed2 swaheed2 commented Sep 7, 2016

I believe the DEFAULT_CHANGES_TIMEOUT was supposed to be 60 seconds as mentioned in the f288ed8 for 1.3.1 and in the commit message.

I believe the DEFAULT_CHANGES_TIMEOUT was supposed to be 60 seconds as mentioned in the couchbase@f288ed8 and in the comment.
@ajres ajres added the review label Sep 7, 2016
@hideki
Copy link
Copy Markdown

hideki commented Sep 7, 2016

HI @swaheed2,
Thank you for submitting PR.
Have you ever singed Couchbase Contributor License Agreement?
If not, can you sign it?
Thanks!
Hideki

@hideki hideki self-assigned this Sep 7, 2016
@hideki hideki added this to the 1.4.0 milestone Sep 7, 2016
@swaheed2
Copy link
Copy Markdown
Contributor Author

swaheed2 commented Sep 7, 2016

yes I've signed it @hideki .

@hideki
Copy link
Copy Markdown

hideki commented Sep 7, 2016

@swaheed2 thank you for your quick response. I will merge this as soon as we release 1.3.1.

@hideki hideki closed this Sep 7, 2016
@hideki hideki reopened this Sep 7, 2016
@pasin pasin modified the milestones: 1.3.1, 1.4.0 Sep 7, 2016
@pasin pasin merged commit 44449cb into couchbase:master Sep 7, 2016
@pasin pasin removed the review label Sep 7, 2016
@pasin
Copy link
Copy Markdown
Contributor

pasin commented Sep 7, 2016

@swaheed2 Thanks for catching the mistake.

@swaheed2 swaheed2 deleted the patch-2 branch September 7, 2016 22:14
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.

4 participants