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

feat: add support for displaying and installing uninstalled themes #648

Merged
merged 2 commits into from
Oct 18, 2022

Conversation

ruibaby
Copy link
Member

@ruibaby ruibaby commented Oct 17, 2022

What type of PR is this?

/kind feature
/milestone 2.0

What this PR does / why we need it:

支持显示和安装未安装的主题,以方便主题开发的时候,创建主题资源。适配 halo-dev/halo#2586

Which issue(s) this PR fixes:

Fixes halo-dev/halo#2554

Screenshots:

image

Special notes for your reviewer:

/cc @halo-dev/sig-halo-console

测试方式:

  1. 需要 pnpm install
  2. Halo 需要切换到 feat: add an API to list uninstalled themes halo#2586 PR 的分支。
  3. 在本地的 ~/halo-dev/themes 创建新的主题,需要包含 themes.yaml,或者将现有的主题直接下载到 ~/halo-dev/themes
  4. 检查后台主题管理的主题列表中是否显示了未安装的主题,以及测试是否可以安装成功。

Does this PR introduce a user-facing change?

支持显示和安装未安装的主题,以方便主题开发的时候,创建主题资源。

@f2c-ci-robot f2c-ci-robot bot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. release-note Denotes a PR that will be considered when it comes time to generate release notes. labels Oct 17, 2022
@f2c-ci-robot f2c-ci-robot bot added this to the 2.0 milestone Oct 17, 2022
@f2c-ci-robot f2c-ci-robot bot added the kind/feature Categorizes issue or PR as related to a new feature. label Oct 17, 2022
@vercel
Copy link

vercel bot commented Oct 17, 2022

@ruibaby is attempting to deploy a commit to the halo-dev Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Oct 17, 2022

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

Name Status Preview Updated
halo-admin-next ✅ Ready (Inspect) Visit Preview Oct 18, 2022 at 10:07AM (UTC)
halo-admin-ui ✅ Ready (Inspect) Visit Preview Oct 18, 2022 at 10:07AM (UTC)

@ruibaby
Copy link
Member Author

ruibaby commented Oct 17, 2022

/hold

等待 #647 合并。

@ruibaby
Copy link
Member Author

ruibaby commented Oct 17, 2022

/retitle feat: add support for displaying and installing uninstalled themes

/unhold

@f2c-ci-robot f2c-ci-robot bot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 17, 2022
@f2c-ci-robot f2c-ci-robot bot changed the title WIP: feat: add support for displaying and installing uninstalled themes feat: add support for displaying and installing uninstalled themes Oct 17, 2022
@f2c-ci-robot f2c-ci-robot bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 17, 2022
@f2c-ci-robot f2c-ci-robot bot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Oct 18, 2022
@ruibaby
Copy link
Member Author

ruibaby commented Oct 18, 2022

/cc @halo-dev/sig-halo-console

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.

建议将“已安装”和“未安装”请求分开。“未安装”主题列表接口请求频次远低于“已安装”。

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.

/approve

@f2c-ci-robot
Copy link

f2c-ci-robot bot commented Oct 18, 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 Oct 18, 2022
@ruibaby
Copy link
Member Author

ruibaby commented Oct 18, 2022

ping @halo-dev/sig-halo-console

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

@f2c-ci-robot f2c-ci-robot bot added the lgtm Indicates that a PR is ready to be merged. label Oct 18, 2022
@f2c-ci-robot f2c-ci-robot bot merged commit dd17087 into halo-dev:main Oct 18, 2022
@ruibaby ruibaby deleted the feat/install-uninstalled-theme branch October 18, 2022 10:17
JohnNiang pushed a commit to JohnNiang/halo that referenced this pull request Mar 2, 2023
…alo-dev/console#648)

#### What type of PR is this?

/kind feature
/milestone 2.0

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

支持显示和安装**未安装**的主题,以方便主题开发的时候,创建主题资源。适配 halo-dev#2586

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

Fixes halo-dev#2554

#### Screenshots:

<img width="1663" alt="image" src="https://user-images.githubusercontent.com/21301288/196148567-f43b1bf3-e745-4c1a-950d-65899c1ae73c.png">

#### Special notes for your reviewer:

/cc @halo-dev/sig-halo-console 

测试方式:

1. 需要 `pnpm install`
2. Halo 需要切换到 halo-dev#2586 PR 的分支。
3. 在本地的 `~/halo-dev/themes` 创建新的主题,需要包含 `themes.yaml`,或者将现有的主题直接下载到 `~/halo-dev/themes`
4. 检查后台主题管理的主题列表中是否显示了未安装的主题,以及测试是否可以安装成功。

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

```release-note
支持显示和安装未安装的主题,以方便主题开发的时候,创建主题资源。
```
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
3 participants