Opened 6 years ago

Last modified 2 years ago

#16280 assigned task

dojox/socket clean up API names

Reported by: Mike Wilcox Owned by: Kris Zyp
Priority: low Milestone: 1.15
Component: IO Version: 1.8.1
Keywords: Cc:
Blocked By: Blocking:

Description

dojox/socket is a constructor, so it's confusing to write:

require(['dojox/socket'], function(Socket){
var socket = new Socket();
});

Likewise, there is a dojox/socket::LongPoll?. In spite of the capital, it is a function not a constructor.

Change History (8)

comment:1 Changed 6 years ago by Mike Wilcox

Component: IODojoX Cometd
Milestone: tbd1.9
Owner: changed from Bryan Forbes to Kris Zyp
Status: newassigned

comment:2 Changed 6 years ago by dylan

Component: DojoX CometdIO

dojox/socket is not dojox/cometd

comment:3 Changed 6 years ago by dylan

Milestone: 1.91.10

comment:4 Changed 5 years ago by Kris Zyp

It would be nice to change these, but this isn't a backwards compatible is it? A more comprehensive fix might be to move this out to its own package, perhaps?

comment:5 Changed 5 years ago by Kris Zyp

Priority: undecidedlow

comment:6 Changed 5 years ago by dylan

Milestone: 1.101.11

Bryan had started to create a new package called tube, https://github.com/bryanforbes/tube

Perhaps we can use that?

comment:7 Changed 3 years ago by dylan

Milestone: 1.111.12

comment:8 Changed 2 years ago by dylan

Milestone: 1.131.15

Ticket planning... move current 1.13 tickets out to 1.15 to make it easier to move tickets into the 1.13 milestone.

Note: See TracTickets for help on using tickets.