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

DDP.connect options are not documented #3837

Closed
ghost opened this issue Feb 28, 2015 · 5 comments · Fixed by #12169
Closed

DDP.connect options are not documented #3837

ghost opened this issue Feb 28, 2015 · 5 comments · Fixed by #12169
Assignees
Labels
confirmed We want to fix or implement it good first issue Good first issue or something that should is nice to do. Project:DDP Project:Docs Type:Feature

Comments

@ghost
Copy link

ghost commented 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

@glasser
Copy link
Contributor

glasser commented Mar 3, 2015

Thanks for pointing out an underdocumented aspect of Meteor! We'll track this here as a doc feature request. Pull requests to improve our docs are always welcome!

@abernix
Copy link
Contributor

abernix commented May 20, 2016

Moved to meteor/docs#22!

@abernix abernix closed this as completed May 20, 2016
@jankapunkt
Copy link
Contributor

@filipenevola I think this should be reopened, the issue remains and docs are now part of this repo again.

@zodern zodern reopened this Feb 16, 2022
@Grubba27
Copy link
Contributor

Hey, @zodern is there any status on this? probably will be closing this and some others by end of the month

@zodern
Copy link
Member

zodern commented Aug 25, 2022

The docs for the options argument of DDP.connect are still missing: https://docs.meteor.com/api/connections.html#DDP-connect

@Grubba27 Grubba27 added the good first issue Good first issue or something that should is nice to do. label Aug 25, 2022
@fredmaiaarantes fredmaiaarantes added the confirmed We want to fix or implement it label Aug 31, 2022
@fredmaiaarantes fredmaiaarantes self-assigned this Aug 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
confirmed We want to fix or implement it good first issue Good first issue or something that should is nice to do. Project:DDP Project:Docs Type:Feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants