Skip to content

Conversation

@soralee
Copy link
Contributor

@soralee soralee commented Feb 8, 2017

What is this PR for?

The home screen is a notebook for use as homepage.
It is the zeppelin.notebook.homescreen variable in conf/zeppelin-site.xml or conf/zeppelin-env.sh.
But currently zeppelin.notebook.homescreen variable doesn't work on shiro until refreshing window.

What type of PR is it?

[Bug Fix]

What is the Jira issue?

How should this be tested?

  1. Set zeppelin.notebook.homescreen variable in conf/zeppelin-site.xml or conf/zeppelin-env.sh.
  2. Turn on shiro.
  3. Start zeppelin-daemon.
  4. Login your account and make sure whether homepage is changed or not.

Screenshots (if appropriate)

[ Before ]
z_2072_b

[ After ]
z_2072_a

Questions:

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

@Leemoonsoo
Copy link
Member

Thanks @soralee for the fix!
Tested and LGTM. CI failure looks not related.

Merge to master if no further discussions.

@soralee
Copy link
Contributor Author

soralee commented Feb 10, 2017

Thanks for taking look @Leemoonsoo 👍

@asfgit asfgit closed this in 896e19e Feb 11, 2017
asfgit pushed a commit that referenced this pull request Feb 11, 2017
### What is this PR for?
The home screen is a notebook for use as homepage.
It is the zeppelin.notebook.homescreen variable in `conf/zeppelin-site.xml` or `conf/zeppelin-env.sh`.
But currently zeppelin.notebook.homescreen variable doesn't work on shiro until refreshing window.

### What type of PR is it?
[Bug Fix]

### What is the Jira issue?
* [ZEPPELIN-2072](https://issues.apache.org/jira/browse/ZEPPELIN-2072)

### How should this be tested?
1. Set zeppelin.notebook.homescreen variable in `conf/zeppelin-site.xml` or `conf/zeppelin-env.sh`.
2. Turn on shiro.
3. Start zeppelin-daemon.
4. Login your account and make sure whether homepage is changed or not.

### Screenshots (if appropriate)
[ Before ]
![z_2072_b](https://cloud.githubusercontent.com/assets/8110458/22730568/dd60512e-ee2a-11e6-8e7e-f379b1eb0e58.gif)

[ After ]
![z_2072_a](https://cloud.githubusercontent.com/assets/8110458/22730590/04482604-ee2b-11e6-9db6-d2656b252274.gif)

### Questions:
* Does the licenses files need update? No
* Is there breaking changes for older versions? No
* Does this needs documentation? No

Author: soralee <[email protected]>

Closes #1990 from soralee/ZEPPELIN-2072_homescreen_shiro and squashes the following commits:

35625c4 [soralee] fix homescreen on shiro

(cherry picked from commit 896e19e)
Signed-off-by: Lee moon soo <[email protected]>
Sign up for free to 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.

2 participants