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

fix: fix page orderview no order #5980

Merged

Conversation

tomatopunk
Copy link

What this PR does / why we need it:

Which issue(s) this PR fixes:

  • Fixes #your-issue_number
  • [Erda Cloud Issue Link](paste your link here)

Specified Reviewers:

/assign @sfwn

ChangeLog

Language Changelog
🇺🇸 English fix page orderview no order
🇨🇳 中文 修复 page orderview 无序

Need cherry-pick to release versions?

Add comment like /cherry-pick release/1.0 when this PR is merged.

For details on the cherry pick process, see the cherry pick requests section under CONTRIBUTING.md.

@erda-bot erda-bot requested a review from sfwn July 14, 2023 02:25
@tomatopunk
Copy link
Author

cc @chengjoey

@codecov
Copy link

codecov bot commented Jul 14, 2023

Codecov Report

Merging #5980 (37d0126) into master (7e7a4e3) will increase coverage by 0.01%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5980      +/-   ##
==========================================
+ Coverage   22.19%   22.20%   +0.01%     
==========================================
  Files        1880     1880              
  Lines      208480   208466      -14     
==========================================
+ Hits        46277    46297      +20     
+ Misses     156910   156877      -33     
+ Partials     5293     5292       -1     

see 6 files with indirect coverage changes

@sfwn
Copy link
Member

sfwn commented Jul 25, 2023

/approve

@erda-bot erda-bot merged commit 0aa512a into erda-project:master Jul 25, 2023
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants