Skip to content

[ZEPPELIN-799] Modify index.html and karma.conf.js for mode-python - #851

Closed
cyhsutw wants to merge 1 commit into
apache:masterfrom
cyhsutw:update-template-files
Closed

[ZEPPELIN-799] Modify index.html and karma.conf.js for mode-python#851
cyhsutw wants to merge 1 commit into
apache:masterfrom
cyhsutw:update-template-files

Conversation

@cyhsutw

Copy link
Copy Markdown
Contributor

What is this PR for?

Python edit mode was added in #825 thank to @astroshim.

I think we'll need to update the src/index.html and test/karma.conf.js under zeppelin-web accordingly to avoid unstaged changes after building.

What type of PR is it?

Improvement

Todos

None

What is the Jira issue?

Probably a subtask of
https://issues.apache.org/jira/browse/ZEPPELIN-799

How should this be tested?

Shouldn't be any since #825 is merged into master and has been tested.

Screenshots (if appropriate)

N/A

Questions:

  • Does the licenses files need update? no
  • Is there breaking changes for older versions? no
  • Does this needs documentation? no

/cc @astroshim

@cyhsutwcyhsutw closed this Apr 22, 2016
@cyhsutwcyhsutw reopened this Apr 22, 2016
@prabhjyotsingh

Copy link
Copy Markdown
Contributor

LGTM

@felixcheung

Copy link
Copy Markdown
Member

right, thanks.

@prabhjyotsingh

Copy link
Copy Markdown
Contributor

Merging if there's no more discussion

@cyhsutw
cyhsutw deleted the update-template-files branch April 24, 2016 21:58
onkarshedge pushed a commit to onkarshedge/incubator-zeppelin that referenced this pull request May 11, 2016
### What is this PR for?
Python edit mode was added in apache#825 thank to astroshim.
I think we'll need to update the `src/index.html` and `test/karma.conf.js` under `zeppelin-web` accordingly to avoid unstaged changes after building.
### What type of PR is it?
Improvement
### Todos
None
### What is the Jira issue?
Probably a subtask of
https://issues.apache.org/jira/browse/ZEPPELIN-799
### How should this be tested?
Shouldn't be any since apache#825 is merged into `master` and has been tested.
### Screenshots (if appropriate)
N/A
### Questions:
* Does the licenses files need update? *no*
* Is there breaking changes for older versions? *no*
* Does this needs documentation? *no*
/cc astroshim
Author: Cheng-Yu Hsu <m@cyhsu.me>
Closesapache#851 from cyhsutw/update-template-files and squashes the following commits:
24901cc [Cheng-Yu Hsu] add `mode-python` to generated html and js
Sign up for freeto 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.

3 participants

@cyhsutw@prabhjyotsingh@felixcheung