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

Image Rendering: Remove PhantomJS support #23460

Merged
merged 13 commits into from
Apr 15, 2020
Merged

Image Rendering: Remove PhantomJS support #23460

merged 13 commits into from
Apr 15, 2020

Conversation

marefr
Copy link
Contributor

@marefr marefr commented Apr 9, 2020

What this PR does / why we need it:
Removes PhantomJS

Which issue(s) this PR fixes:
Closes #13802

Special notes for your reviewer:
Improve UX for share panel/direct link rendered image when there are no renderer available/installed will be handled in a follow up PR.

marefr added 2 commits April 9, 2020 10:42
Removes PhantomJS.
Remove direct link rendered image e2e smoke test for now.

Ref #13802
@marefr marefr added this to the 7.0 milestone Apr 9, 2020
@marefr marefr marked this pull request as ready for review April 15, 2020 16:41
@marefr marefr requested review from oddlittlebird and a team as code owners April 15, 2020 16:41
@marefr marefr requested review from kylebrandt, aknuds1, torkelo and bergquist and removed request for a team April 15, 2020 16:41
@marefr
Copy link
Contributor Author

marefr commented Apr 15, 2020

For reviewers, #23375 and #23593 have already been merged into this PR. This PR adds some custom docker build fixes and breaking change and upgrading notes. Idea is to handle required UX changes in a follow up PR.

Copy link
Contributor

@oddlittlebird oddlittlebird left a comment

Choose a reason for hiding this comment

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

Please edit the Image rendering topic as well.

CHANGELOG.md Outdated Show resolved Hide resolved
docs/sources/installation/upgrading.md Outdated Show resolved Hide resolved
marefr and others added 3 commits April 15, 2020 19:42
Co-Authored-By: Diana Payton <52059945+oddlittlebird@users.noreply.github.com>
@marefr marefr requested a review from oddlittlebird April 15, 2020 19:21
Copy link
Contributor

@oddlittlebird oddlittlebird left a comment

Choose a reason for hiding this comment

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

LGTM!

@marefr marefr merged commit 6e313e7 into master Apr 15, 2020
@marefr marefr deleted the feat_remove_phantomjs branch April 15, 2020 20:17
Copy link
Contributor

@aknuds1 aknuds1 left a comment

Choose a reason for hiding this comment

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

LGTM!

bergquist added a commit to bergquist/grafana that referenced this pull request Apr 16, 2020
* master:
  Linting fixes for provisoning (grafana#23600)
  Image Rendering: Remove PhantomJS support (grafana#23460)
  Storybook: Organizing (grafana#23572)
  @grafana/ui: Remove ColorPallete component (grafana#23592)
bergquist added a commit that referenced this pull request Apr 17, 2020
* master: (139 commits)
  SAML Role and Team sync (open source part) (#23391)
  Transformers: fixes column names generated by seriesToColumn transformer (#23642)
  Table: add a select option when multiple tables exist (#23545)
  Tabs: adds counter component (#23548)
  UX: Minor style tweaks and fixes (#23614)
  Modals: Fixed alignment in header (#23618)
  revent optional event arg change (#23622)
  Build: Stricter linting should break the build (#23606)
  Docs: update users API (#23570)
  Explore: Prevents sizing issue when split (#23596)
  @grafana/ui: Various smaller Icon updates (#23588)
  Because alpine uses musl instead of libc, the e2e/cypress was not compatible (#23602)
  Backend Plugins: Discard logged messages from logger named stdio (#23601)
  Fixed share json (#23607)
  Linting fixes for provisoning (#23600)
  Image Rendering: Remove PhantomJS support (#23460)
  Storybook: Organizing (#23572)
  @grafana/ui: Remove ColorPallete component (#23592)
  Docs: Change header levels of last three sections (#23587)
  CircleCI: Upgrade the build pipeline tool (#23595)
  ...
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.

Remove PhantomJS
3 participants