Skip to content
This repository has been archived by the owner on Oct 24, 2021. It is now read-only.

DDP.connect options parameter #22

Closed
abernix opened this issue May 20, 2016 · 3 comments
Closed

DDP.connect options parameter #22

abernix opened this issue May 20, 2016 · 3 comments
Labels

Comments

@abernix
Copy link
Contributor

abernix commented May 20, 2016

DDP.connect is documented, but options are omitted. I realized just after I moved this that this is because they're not in the jsdoc definition.

@sanjo commented on Sat Feb 28 2015

You can pass options when creating a DDP connection with DDP.connect. But currently those are not documented and I had to dig through the code. I'm not sure if this is intentional, so I open this issue.

Available options can be found here: https://github.com/meteor/meteor/blob/devel/packages/ddp/livedata_connection.js#L29-L64

@abernix abernix changed the title DDP.connect options are not documented DDP.connect options parameter May 20, 2016
@lorensr lorensr added the bug label May 22, 2016
@lorensr
Copy link
Contributor

lorensr commented May 22, 2016

4 options, 1 private. Someone could want to give sockjs options, but we probably shouldn't document any underscored vars?

image

@tmeasday
Copy link
Contributor

I guess not, yeah. TBH not sure why it's underscored, but hey.

@filipenevola
Copy link
Collaborator

I'm closing this issue because it's too old.

If you think this issue is still relevant please open a new one.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants