Skip to content

Conversation

@yinxulai
Copy link
Collaborator

No description provided.

@codecov
Copy link

codecov bot commented Nov 23, 2021

Codecov Report

Merging #543 (f082f47) into master (fa78fde) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #543   +/-   ##
=======================================
  Coverage   74.39%   74.39%           
=======================================
  Files          21       21           
  Lines        1117     1117           
  Branches      210      210           
=======================================
  Hits          831      831           
  Misses        286      286           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fa78fde...f082f47. Read the comment docs.

nighca
nighca previously approved these changes Nov 23, 2021
Copy link
Contributor

@nighca nighca left a comment

Choose a reason for hiding this comment

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

👓

import qiniuLogo from './assets/logo.svg'
import classnames from './style.less'


Copy link
Contributor

Choose a reason for hiding this comment

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

NIP: useless blank line

import classnames from './style.less'


function OfficialSite() {
Copy link
Contributor

Choose a reason for hiding this comment

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

NIP:

Suggested change
function OfficialSite() {
function Copyright() {

Copy link
Contributor

@nighca nighca left a comment

Choose a reason for hiding this comment

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

📈

@yinxulai yinxulai merged commit 3f6632f into qiniu:master Nov 23, 2021
Comment on lines +43 to +44
* [JavaScript SDK 在线示例 V3](https://js-sdk-demo.qiniu.io)
* [JavaScript SDK 在线示例 V2](http://jssdk-v2.demo.qiniu.io)
Copy link
Collaborator

Choose a reason for hiding this comment

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

这 v2 v3 语义不是很明确,一开始根本想不出来是 js sdk 的版本… 还以为是 demo 的版本 = =
文档开头说的 2.x 3.x 这里怎么就成了 v2 v3 术语不配套?
而且对这个文档来说,v3 本来就是配套的,v2 才是奇怪的那个… 比如上面就有 v2 的文档入口,这个为什么是例外

target="_blank"
href="http://jssdk-v2.demo.qiniu.io"
>
V2版本
Copy link
Collaborator

Choose a reason for hiding this comment

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

术语统一 & 空格

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants