Skip to content

Release 1.0.0 - #293

Merged
wangxiao merged 1 commit into
leancloud:masterfrom
wangxiao:node
May 30, 2016
Merged

Release 1.0.0#293
wangxiao merged 1 commit into
leancloud:masterfrom
wangxiao:node

Conversation

@wangxiao

Copy link
Copy Markdown
Contributor

No description provided.

@codecov-io

codecov-io commented May 30, 2016

Copy link
Copy Markdown

Current coverage is 63.87%

Merging #293 into master will not change coverage

  1. 16 files in dist/node were modified. more
    • Misses -6
    • Hits +20
@@             master       #293   diff @@
==========================================
  Files            27         27          
  Lines          2898       2898          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           1851       1851          
  Misses         1047       1047          
  Partials          0          0          

Powered by Codecov. Last updated by 7d8b430...48facb1

Comment thread changelog.md Outdated
@@ -1,3 +1,12 @@
# 1.0.0 日期:2016 年 5 月 30 日
* 服务器针对浏览器 CORS 返回 307 无法跳转的问题,采用通过返回 410 和自定义字段的方式,在 SDK 层面支持重定向 request

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

changelog 是给用户看的,应该写对用户来说有用的信息,尤其是 API 的变动。我们其他工具使用了一些内部的 API 是另一个问题,但是还是不应该把内部的重构写给用户看。

  • 修正 disableCurrentUser 时,内部错误的调用 currentUser 相关接口(修复的影响用户的具体问题是什么?)
  • 弃用 AV.Error 对象,改为内部模块(需要说明如果之前用户使用了 AV.Error 现在应该怎么做)

不需要写的 log 有:

  • 服务器针对浏览器 CORS 返回 307 无法跳转的问题,采用通过返回 410 和自定义字段的方式,在 SDK 层面支持重定向 request
  • 增加内部接口 AV._config.userAgent
  • 弃用内部接口 AV._request
  • 移除 AV.applicationProduction 改为 AV._config.applicationProduction 内部接口

@wangxiao
wangxiao merged commit 6afac1d into leancloud:master May 30, 2016
@wangxiao
wangxiao deleted the node branch May 30, 2016 12:33
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.

5 participants