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

Remove unwanted openstack versions from chart-data.js #12848

Merged
merged 1 commit into from
May 10, 2023
Merged

Conversation

petesfrench
Copy link
Contributor

@petesfrench petesfrench commented May 10, 2023

Done

  • Remove 'Rock, Stein, Train, Ussuri, Victoria, Xena' from the chart based on Tytus' comments

QA

Issue / Card

Fixes https://warthogs.atlassian.net/browse/WD-3155

@webteam-app
Copy link

Demo starting at https://ubuntu-com-12848.demos.haus

@codecov
Copy link

codecov bot commented May 10, 2023

Codecov Report

Merging #12848 (07b08ee) into main (34f3051) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main   #12848   +/-   ##
=======================================
  Coverage   73.67%   73.67%           
=======================================
  Files         104      104           
  Lines        2708     2708           
  Branches      840      840           
=======================================
  Hits         1995     1995           
  Misses        685      685           
  Partials       28       28           

Copy link
Contributor

@mtruj013 mtruj013 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just gotta remove them from the table on medium screens as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants