Skip to content
This repository has been archived by the owner on Jan 21, 2024. It is now read-only.

refactor: add validation for post and singlePage settings form #791

Merged
merged 2 commits into from
Dec 24, 2022

Conversation

ruibaby
Copy link
Member

@ruibaby ruibaby commented Dec 21, 2022

What type of PR is this?

/kind improvement

What this PR does / why we need it:

重构文章和自定义页面的设置表单,支持提交时验证表单。

因为之前的多选项卡设计导致无法同时验证所有表单,所以这个 PR 重构了表单的布局。

Which issue(s) this PR fixes:

Fixes halo-dev/halo#2859

Screenshots:

2022-12-21 17 23 22

Special notes for your reviewer:

测试方式:

  1. 测试在内容编辑页面和列表打开文章和自定义页面的设置表单。
  2. 检查表单验证是否有效。

Does this PR introduce a user-facing change?

重构 Console 端文章和自定义页面的设置表单布局,支持提交时验证表单。

@f2c-ci-robot f2c-ci-robot bot added the release-note Denotes a PR that will be considered when it comes time to generate release notes. label Dec 21, 2022
@vercel
Copy link

vercel bot commented Dec 21, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
ui ✅ Ready (Inspect) Visit Preview Dec 22, 2022 at 8:54AM (UTC)

@f2c-ci-robot f2c-ci-robot bot added the kind/improvement Categorizes issue or PR as related to a improvement. label Dec 21, 2022
@JohnNiang
Copy link
Member

如果该问题在 2.0.x 中也存在,建议专门提交一个修复 bug 的 PR,并 patch 到 release-2.0 分支。

@ruibaby
Copy link
Member Author

ruibaby commented Dec 23, 2022

如果该问题在 2.0.x 中也存在,建议专门提交一个修复 bug 的 PR,并 patch 到 release-2.0 分支。

就是为了修改这个 bug 才这样修改。如果不重构来修复这个问题,改动可能会更大。

Copy link
Member

@guqing guqing left a comment

Choose a reason for hiding this comment

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

/lgtm
As expected 🎉

@f2c-ci-robot f2c-ci-robot bot added the lgtm Indicates that a PR is ready to be merged. label Dec 23, 2022
Copy link
Member

@JohnNiang JohnNiang left a comment

Choose a reason for hiding this comment

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

It works very well.
/lgtm

@f2c-ci-robot
Copy link

f2c-ci-robot bot commented Dec 24, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: JohnNiang

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

The pull request process is described 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 the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Dec 24, 2022
@f2c-ci-robot f2c-ci-robot bot merged commit f7ece91 into halo-dev:main Dec 24, 2022
JohnNiang pushed a commit to JohnNiang/halo that referenced this pull request Mar 2, 2023
…dev/console#791)

#### What type of PR is this?

/kind improvement

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

重构文章和自定义页面的设置表单,支持提交时验证表单。

> 因为之前的多选项卡设计导致无法同时验证所有表单,所以这个 PR 重构了表单的布局。

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

Fixes halo-dev#2859

#### Screenshots:

![2022-12-21 17 23 22](https://user-images.githubusercontent.com/21301288/208870059-5039a565-def2-4622-9a78-de30dceb4d65.gif)

#### Special notes for your reviewer:

测试方式:

1. 测试在内容编辑页面和列表打开文章和自定义页面的设置表单。
2. 检查表单验证是否有效。

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


```release-note
重构 Console 端文章和自定义页面的设置表单布局,支持提交时验证表单。
```
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. kind/improvement Categorizes issue or PR as related to a improvement. lgtm Indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

文章设置在提交的时候没有进行表单验证
3 participants