Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

添加冬日绘版系列内容 #13

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

Merged
0f-0b merged 3 commits into 0f-0b:master from EarthMessenger:master
Jan 16, 2022
Merged

添加冬日绘版系列内容 #13

0f-0b merged 3 commits into 0f-0b:master from EarthMessenger:master
Jan 16, 2022

Conversation

@EarthMessenger
Copy link
Contributor

@EarthMessenger EarthMessenger commented Jan 3, 2022

No description provided.

Copy link
Owner

@0f-0b 0f-0b left a comment

Choose a reason for hiding this comment

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

感谢 PR。有几点建议希望能考虑。

misc.md Outdated
</tr>
<tr>
<th align="right">响应主体</th>
<td><code>text/plain</code></td>
Copy link
Owner

Choose a reason for hiding this comment

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

详细说明数据格式。

misc.md Outdated
<td><code>application/json</code>(<code>{ token: string; }</code>)</td>
</tr>
</table>
注:不需要 CSRF 令牌。
Copy link
Owner

Choose a reason for hiding this comment

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

不需要特别说明。

misc.md Outdated
<table>
<tr>
<th align="right">请求</th>
<td><code>POST /paintboard/resetToken </code></td>
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
<td><code>POST /paintboard/resetToken</code></td>
<td><code>POST /paintboard/resetToken</code></td>

misc.md Outdated
</tr>
<tr>
<th align="right">响应主体</th>
<td><code>application/json</code>(<code>{ token: string; }</code>)</td>
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
<td><code>application/json</code>(<code>{ token: string; }</code>)</td>
<td><code>application/json</code>(<code>{ token: string; }</code>)</td>

misc.md Outdated
Comment on lines 116 to 128
## 冬日绘版更新检测

<table>
<tr>
<th align="right">请求</th>
<td><code>GET wss://ws.luogu.com.cn/ws </code></td>
</tr>
<tr>
<th align="right">响应主体</th>
<td>自己意会</td>
</tr>
</table>

Copy link
Owner

Choose a reason for hiding this comment

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

关于 WS 的内容可以暂时不加。加上时是要详细说明通信协议的(虽然就是,感觉怎么说都不如直接看代码实现清楚

misc.md Outdated
<table>
<tr>
<th align="right">请求</th>
<td><code>POST /paintboard/paint </code></td>
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
<td><code>POST /paintboard/paint</code></td>
<td><code>POST /paintboard/paint</code></td>

misc.md Outdated
</tr>
<tr>
<th align="right">请求主体</th>
<td><code>application/json</code>(<code>{ x: int; y: int; color: char; }</code>)</td>
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
<td><code>application/json</code>(<code>{ x: int; y: int; color: char; }</code>)</td>
<td><code>application/json</code>(<code>{ x: number; y: number; color: number; }</code>)</td>

类型用 TypeScript 说明。

misc.md Outdated
</tr>
<tr>
<th align="right">响应主体</th>
<td><code>application/json</code>(<code>{}</code>)</td>
Copy link
Owner

Choose a reason for hiding this comment

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

Suggested change
<td><code>application/json</code>(<code>{}</code>)</td>
<td><code>application/json</code>(<code>{}</code>)</td>

misc.md Outdated
</tr>
</table>

## 冬日绘版更新 token
Copy link
Owner

Choose a reason for hiding this comment

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

或许可以称为"重置 token"。

@0f-0b 0f-0b merged commit b7c989a into 0f-0b:master Jan 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@0f-0b 0f-0b 0f-0b approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /