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

feat: record the comment query conditions in the route query parameters #4209

Merged
merged 2 commits into from Jul 21, 2023

Conversation

ruibaby
Copy link
Member

@ruibaby ruibaby commented Jul 13, 2023

What type of PR is this?

/area console
/kind feature
/milestone 2.8.x

What this PR does / why we need it:

在评论数据管理列表页面路由中记录查询条件,包括分页信息、筛选信息等。可以保证在刷新浏览器窗口或者从其他页面返回的时候不丢失筛选条件。

image

Special notes for your reviewer:

需要测试:

  1. 评论管理列表的所有筛选项是否可以正常工作。
  2. 尝试设置部分筛选,然后刷新页面,观察筛选条件是否正常保留。

Does this PR introduce a user-facing change?

Console 端的评论管理列表支持在地址栏记录筛选条件。

@f2c-ci-robot f2c-ci-robot bot added release-note Denotes a PR that will be considered when it comes time to generate release notes. area/console Issues or PRs related to the Halo Console labels Jul 13, 2023
@f2c-ci-robot f2c-ci-robot bot added this to the 2.8.x milestone Jul 13, 2023
@f2c-ci-robot f2c-ci-robot bot requested review from lan-yonghui and LIlGG July 13, 2023 05:25
@f2c-ci-robot f2c-ci-robot bot added the kind/feature Categorizes issue or PR as related to a new feature. label Jul 13, 2023
@codecov
Copy link

codecov bot commented Jul 13, 2023

Codecov Report

Merging #4209 (4539183) into main (efcf526) will increase coverage by 0.16%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #4209      +/-   ##
============================================
+ Coverage     60.04%   60.21%   +0.16%     
- Complexity     2409     2420      +11     
============================================
  Files           366      366              
  Lines         12587    12602      +15     
  Branches        906      907       +1     
============================================
+ Hits           7558     7588      +30     
+ Misses         4590     4573      -17     
- Partials        439      441       +2     

see 10 files with indirect coverage changes

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 Jul 13, 2023
Signed-off-by: Ryan Wang <i@ryanc.cc>
@f2c-ci-robot f2c-ci-robot bot removed the lgtm Indicates that a PR is ready to be merged. label Jul 20, 2023
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.

/lgtm

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

f2c-ci-robot bot commented Jul 21, 2023

[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 Jul 21, 2023
@f2c-ci-robot f2c-ci-robot bot merged commit 7ee6e05 into halo-dev:main Jul 21, 2023
4 checks passed
@ruibaby ruibaby modified the milestones: 2.8.x, 2.8.0 Jul 21, 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. area/console Issues or PRs related to the Halo Console 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.

None yet

3 participants