Skip to content

Fix renderComplete backport#9372

Merged
thomasneirynck merged 1 commit intoelastic:5.1from
thomasneirynck:fix-renderComplete-backport
Dec 5, 2016
Merged

Fix renderComplete backport#9372
thomasneirynck merged 1 commit intoelastic:5.1from
thomasneirynck:fix-renderComplete-backport

Conversation

@thomasneirynck
Copy link
Copy Markdown
Contributor

Fixes divergence between master vs 5.x/5.1

Backport of renderComplete change (elastic#9183) happened before Tagcloud was backported to 5.x/5.1. This caused a divergence in master vs 5.x/5.1 in the tag-cloud code.
@w33ble
Copy link
Copy Markdown
Contributor

w33ble commented Dec 5, 2016

LGTM!

@ppisljar
Copy link
Copy Markdown
Contributor

ppisljar commented Dec 5, 2016

LGTM

@thomasneirynck thomasneirynck merged commit 75d284a into elastic:5.1 Dec 5, 2016
elastic-jasper added a commit that referenced this pull request Dec 5, 2016
Backports PR #9372

**Commit 1:**
Fix renderComplete backport

Backport of renderComplete change (#9183) happened before Tagcloud was backported to 5.x/5.1. This caused a divergence in master vs 5.x/5.1 in the tag-cloud code.

* Original sha: 0fa4808
* Authored by Thomas Neirynck <thomas@elastic.co> on 2016-12-05T18:29:04Z
thomasneirynck pushed a commit that referenced this pull request Dec 5, 2016
Backports PR #9372

**Commit 1:**
Fix renderComplete backport

Backport of renderComplete change (#9183) happened before Tagcloud was backported to 5.x/5.1. This caused a divergence in master vs 5.x/5.1 in the tag-cloud code.

* Original sha: 0fa4808
* Authored by Thomas Neirynck <thomas@elastic.co> on 2016-12-05T18:29:04Z
airow pushed a commit to airow/kibana that referenced this pull request Feb 16, 2017
Backport of renderComplete change (elastic#9183) happened before Tagcloud was backported to 5.x/5.1. This caused a divergence in master vs 5.x/5.1 in the tag-cloud code.

Former-commit-id: 75d284a
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.

4 participants