Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pref: improve the structure of ContributorVo (#4645) #4705

Merged
merged 1 commit into from
Oct 11, 2023
Merged

pref: improve the structure of ContributorVo (#4645) #4705

merged 1 commit into from
Oct 11, 2023

Conversation

AirboZH
Copy link
Member

@AirboZH AirboZH commented Oct 10, 2023

What type of PR is this?

/kind improvement
/kind api-change

What this PR does / why we need it:

优化ContributorVo格式,支持更多信息。

Which issue(s) this PR fixes:

Fixes #4645

Special notes for your reviewer:

none

Does this PR introduce a user-facing change?

contributorFinder 支持返回用户元数据、注册信息、用户权限等。

@f2c-ci-robot f2c-ci-robot bot added kind/improvement Categorizes issue or PR as related to a improvement. release-note Denotes a PR that will be considered when it comes time to generate release notes. kind/api-change Categorizes issue or PR as related to adding, removing, or otherwise changing an API labels Oct 10, 2023
@f2c-ci-robot f2c-ci-robot bot requested review from guqing and minliacom October 10, 2023 05:53
@codecov
Copy link

codecov bot commented Oct 10, 2023

Codecov Report

Merging #4705 (43f61e8) into main (72e5cf2) will decrease coverage by 0.01%.
Report is 1 commits behind head on main.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##               main    #4705      +/-   ##
============================================
- Coverage     61.59%   61.58%   -0.01%     
  Complexity     2797     2797              
============================================
  Files           403      403              
  Lines         14723    14725       +2     
  Branches       1023     1023              
============================================
  Hits           9068     9068              
- Misses         5163     5165       +2     
  Partials        492      492              
Files Coverage Δ
...a/run/halo/app/theme/finders/vo/ContributorVo.java 11.11% <0.00%> (-1.39%) ⬇️

@AirboZH AirboZH requested a review from guqing October 10, 2023 14:46
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 f2c-ci-robot bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 11, 2023
@JohnNiang
Copy link
Member

/milestone 2.11.x

@f2c-ci-robot f2c-ci-robot bot added this to the 2.11.x milestone Oct 11, 2023
@AirboZH AirboZH requested a review from guqing October 11, 2023 07:19
@ruibaby ruibaby changed the title pref: Improve the Structure of ContributorVo (#4645) pref: improve the structure of ContributorVo (#4645) Oct 11, 2023
@ruibaby
Copy link
Member

ruibaby commented Oct 11, 2023

Hi @AirboZH,可能也需要更新一下主题开发文档。

Copy link
Member

@ruibaby ruibaby 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 11, 2023
@f2c-ci-robot
Copy link

f2c-ci-robot bot commented Oct 11, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: guqing, JohnNiang, ruibaby

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:
  • OWNERS [JohnNiang,guqing,ruibaby]

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 merged commit 767aa53 into halo-dev:main Oct 11, 2023
2 checks passed
f2c-ci-robot bot pushed a commit to halo-dev/docs that referenced this pull request Oct 11, 2023
/kind documentation

link: halo-dev/halo#4705


```release-note
更新 ContributorVo 格式
```
@JohnNiang JohnNiang modified the milestones: 2.11.x, 2.11.0 Oct 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. 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. 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.

contributorFinder 支持返回用户元数据
4 participants