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

Use correct filters in progress bar links #4544

Merged
merged 1 commit into from Mar 4, 2022

Conversation

Martchus
Copy link
Contributor

@Martchus Martchus commented Mar 4, 2022

The dashboard's progress bar sections combine certain results. The link
should contain filter parameters for all those results - otherwise the
list of jobs shown in the linked overview page does not match the figure
from the progress bar.

See https://progress.opensuse.org/issues/107878

The dashboard's progress bar sections combine certain results. The link
should contain filter parameters for all those results - otherwise the
list of jobs shown in the linked overview page does not match the figure
from the progress bar.

See https://progress.opensuse.org/issues/107878
@codecov
Copy link

codecov bot commented Mar 4, 2022

Codecov Report

Merging #4544 (8e47091) into master (b5d8be1) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #4544   +/-   ##
=======================================
  Coverage   97.96%   97.96%           
=======================================
  Files         374      374           
  Lines       34170    34170           
=======================================
  Hits        33475    33475           
  Misses        695      695           
Impacted Files Coverage Δ
lib/OpenQA/WebAPI/Plugin/Helpers.pm 99.19% <100.00%> (ø)
t/ui/14-dashboard-parents.t 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b5d8be1...8e47091. Read the comment docs.

@mergify mergify bot merged commit 0703183 into os-autoinst:master Mar 4, 2022
@Martchus Martchus deleted the progress-bar-links branch March 4, 2022 15:25
openqabot pushed a commit to openqabot/openQA that referenced this pull request Mar 5, 2022
commit 0703183
Merge: 4f3a314 8e47091
Author:     mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
AuthorDate: Fri Mar 4 15:24:23 2022 +0000
Commit:     GitHub <noreply@github.com>
CommitDate: Fri Mar 4 15:24:23 2022 +0000

    Merge pull request os-autoinst#4544 from Martchus/progress-bar-links

    Use correct filters in progress bar links
openqabot pushed a commit to openqabot/openQA that referenced this pull request Mar 6, 2022
commit 0703183
Merge: 4f3a314 8e47091
Author:     mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
AuthorDate: Fri Mar 4 15:24:23 2022 +0000
Commit:     GitHub <noreply@github.com>
CommitDate: Fri Mar 4 15:24:23 2022 +0000

    Merge pull request os-autoinst#4544 from Martchus/progress-bar-links

    Use correct filters in progress bar links
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants