Skip to content

Conversation

@EarthMessenger
Copy link
Contributor

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants