-
Notifications
You must be signed in to change notification settings - Fork 19
Conversation
var url = ''; | ||
var protocol = 'http://'; | ||
if (win && win.location.protocol === 'https:') { | ||
protocol = 'https://'; | ||
} | ||
url = protocol + 'router-g0-push.avoscloud.com/v1/route?appId=' + appId ; | ||
var node = ''; | ||
switch (options.country) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
慎用 country
这个名字啊。。。我觉得叫 env
比较好
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
万一过几天开通了香港台湾专区……
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
region
2015-06-02 13:24 GMT+08:00 Ning Sun notifications@github.com:
In src/AV.push.js
#30 (comment):var url = ''; var protocol = 'http://'; if (win && win.location.protocol === 'https:') { protocol = 'https://'; }
url = protocol + 'router-g0-push.avoscloud.com/v1/route?appId=' + appId ;
var node = '';
switch (options.country) {
慎用 country 这个名字啊。。。我觉得叫 env 比较好
—
Reply to this email directly or view it on GitHub
https://github.com/leancloud/js-push-sdk/pull/30/files#r31493577.
break; | ||
case 'us': | ||
options.host = 'avoscloud.us'; | ||
break; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
可以加个 default 扔个异常出来
除了一个 comment 都 LGTM 了,可以合并了 |
已增加,合并了。 |
@sunng87