Skip to content
This repository has been archived by the owner on Feb 17, 2023. It is now read-only.

refactor: structure of the setting spec #2

Merged
merged 1 commit into from
Sep 22, 2022
Merged

Conversation

ruibaby
Copy link
Member

@ruibaby ruibaby commented Sep 21, 2022

修改 settings.yaml 的 spec 定义。适配 halo-dev/halo#2440

/kind improvement
/kind api-change

None

Signed-off-by: Ryan Wang <i@ryanc.cc>
@f2c-ci-robot f2c-ci-robot bot added kind/improvement Categorizes issue or PR as related to a improvement. kind/api-change Categorizes issue or PR as related to adding, removing, or otherwise changing an API do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. labels Sep 21, 2022
@f2c-ci-robot
Copy link

f2c-ci-robot bot commented Sep 21, 2022

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@f2c-ci-robot f2c-ci-robot bot added release-note-none Denotes a PR that doesn't merit a release note. and removed do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. labels Sep 21, 2022
f2c-ci-robot bot pushed a commit to halo-dev/console that referenced this pull request Sep 22, 2022
#### What type of PR is this?

/kind improvement
/milestone 2.0
/kind api-change

#### What this PR does / why we need it:

重构 Setting 模型的结构,适配 halo-dev/halo#2440

todo list:

- [x] 更新 `@halo-dev/api-client`

#### Which issue(s) this PR fixes:

Fixes halo-dev/halo#2414

#### Special notes for your reviewer:

测试方式:

1. 本地 admin 仓库切换到当前 PR 的分支,halo 仓库切换到 halo-dev/halo#2440 PR 的分支。
2. 测试后台各个表单功能是否正常。
3. 主题设置表单可使用 halo-sigs/theme-anatole#2 PR 的分支进行测试。

#### Does this PR introduce a user-facing change?

```release-note
None
```
@ruibaby ruibaby merged commit fcc09df into main Sep 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/api-change Categorizes issue or PR as related to adding, removing, or otherwise changing an API kind/improvement Categorizes issue or PR as related to a improvement. release-note-none Denotes a PR that doesn't merit a release note.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant