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

feat: update theme requires field and supports show failed message #826

Merged
merged 1 commit into from
Jan 19, 2023

Conversation

ruibaby
Copy link
Member

@ruibaby ruibaby commented Jan 17, 2023

What type of PR is this?

/kind feature

What this PR does / why we need it:

修改主题的 spec.require 字段为 spec.requires,以及支持显示错误信息。适配:halo-dev/halo#3150

Which issue(s) this PR fixes:

Fixes halo-dev/halo#3088

Screenshots:

image

image

Special notes for your reviewer:

测试方式:

  1. Halo 切换到 feat: support validate requires version for theme halo#3150 分支。
  2. Console 需要 pnpm install
  3. 修改主题的 spec.requires,根据 feat: support validate requires version for theme halo#3150 中的测试条件修改这个值。
  4. 检查是否可以显示异常版本信息。

Does this PR introduce a user-facing change?

Console 端支持显示主题的版本异常信息

@f2c-ci-robot f2c-ci-robot bot added kind/feature Categorizes issue or PR as related to a new feature. release-note Denotes a PR that will be considered when it comes time to generate release notes. labels Jan 17, 2023
@f2c-ci-robot f2c-ci-robot bot requested review from Aanko and wzrove January 17, 2023 03:04
@vercel
Copy link

vercel bot commented Jan 17, 2023

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

Name Status Preview Comments Updated
ui ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Jan 17, 2023 at 3:05AM (UTC)

Copy link
Member

@wzrove wzrove left a comment

Choose a reason for hiding this comment

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

/approve

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.

image

/lgtm

@f2c-ci-robot f2c-ci-robot bot added the lgtm Indicates that a PR is ready to be merged. label Jan 19, 2023
@f2c-ci-robot
Copy link

f2c-ci-robot bot commented Jan 19, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: JohnNiang, wzrove

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 Jan 19, 2023
@f2c-ci-robot f2c-ci-robot bot merged commit 0fa6909 into halo-dev:main Jan 19, 2023
@ruibaby ruibaby deleted the perf/theme-require branch January 19, 2023 04:45
@ruibaby ruibaby added this to the 2.2.0 milestone Jan 31, 2023
JohnNiang pushed a commit to JohnNiang/halo that referenced this pull request Mar 2, 2023
…alo-dev/console#826)

#### What type of PR is this?

/kind feature

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

修改主题的 `spec.require` 字段为 `spec.requires`,以及支持显示错误信息。适配:halo-dev#3150

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

Fixes halo-dev#3088

#### Screenshots:

<img width="1692" alt="image" src="https://user-images.githubusercontent.com/21301288/212800478-0fe53ae8-4e31-4810-89aa-4b06d1b700fb.png">
<img width="1056" alt="image" src="https://user-images.githubusercontent.com/21301288/212800550-7edd7c0a-564c-4df4-baee-b3ffe6b3352d.png">

#### Special notes for your reviewer:

测试方式:

1. Halo 切换到 halo-dev#3150 分支。
2. Console 需要 `pnpm install`。
3. 修改主题的 `spec.requires`,根据 halo-dev#3150 中的测试条件修改这个值。
4. 检查是否可以显示异常版本信息。

#### 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/feature Categorizes issue or PR as related to a new feature. 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