[GitHub] zeppelin pull request #2669: available_interpreters.png replace.

classic Classic list List threaded Threaded
6 messages Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] zeppelin pull request #2669: available_interpreters.png replace.

anthonycorbacho
GitHub user kimjmin opened a pull request:

    https://github.com/apache/zeppelin/pull/2669

    available_interpreters.png replace.

    ### What is this PR for?
    Change `docs/assets/themes/zeppelin/img/available_interpreters.png` file.
    replace old `elasticsearch` logo to new one.
   
    ### What type of PR is it?
    Documentation
   
    ### Todos
    * [x] - replace old `elasticsearch` logo to new one on `available_interpreters.png` file.
   
    ### What is the Jira issue?
    N/A
   
    ### How should this be tested?
    N/A
   
    ### Screenshots (if appropriate)
   
    ### Questions:
    * Does the licenses files need update? N.
    * Is there breaking changes for older versions? N.
    * Does this needs documentation? N.


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/kimjmin/zeppelin master

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/zeppelin/pull/2669.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #2669
   
----
commit 6134175921d2b2c727a46bc9207ead7c6ce129fc
Author: Jongmin Kim <[hidden email]>
Date:   2017-11-15T08:03:46Z

    available_interpreters.png file change.
   
    Change `docs/assets/themes/zeppelin/img/available_interpreters.png`
    file.
    replace old `elasticsearch` logo to new one.

commit b5e5341011a6fea74a6f0d36bdcbc524756299d2
Author: Jongmin Kim <[hidden email]>
Date:   2017-11-15T08:07:57Z

    Revert "available_interpreters.png file change."
   
    This reverts commit 6134175921d2b2c727a46bc9207ead7c6ce129fc.

commit 349194d9146cb8ce75879f04b4649ba6576a730a
Author: Jongmin Kim <[hidden email]>
Date:   2017-11-15T08:16:04Z

    available_interpreters.png file change.
   
    Change `docs/assets/themes/zeppelin/img/available_interpreters.png`
    file.
    replace old `elasticsearch` logo to new one.

----


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] zeppelin issue #2669: available_interpreters.png replace.

anthonycorbacho
Github user jongyoul commented on the issue:

    https://github.com/apache/zeppelin/pull/2669
 
    Thanks for the contribution. LGTM. Merge it sooner


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] zeppelin issue #2669: available_interpreters.png replace.

anthonycorbacho
In reply to this post by anthonycorbacho
Github user jongyoul commented on the issue:

    https://github.com/apache/zeppelin/pull/2669
 
    @AhyoungRyu Can you help to deploy it to the web?


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] zeppelin pull request #2669: available_interpreters.png replace.

anthonycorbacho
In reply to this post by anthonycorbacho
Github user asfgit closed the pull request at:

    https://github.com/apache/zeppelin/pull/2669


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] zeppelin issue #2669: available_interpreters.png replace.

anthonycorbacho
In reply to this post by anthonycorbacho
Github user jongyoul commented on the issue:

    https://github.com/apache/zeppelin/pull/2669
 
    @kimjmin Can you check the site? I updated it


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] zeppelin issue #2669: available_interpreters.png replace.

anthonycorbacho
In reply to this post by anthonycorbacho
Github user kimjmin commented on the issue:

    https://github.com/apache/zeppelin/pull/2669
 
    @jongyoul
    Thanks. Page looks beautiful. 👍


---