Skip to content

ZEPPELIN-31 fix REST API base path in web-app - #30

Closed
bzz wants to merge 3 commits into
apache:masterfrom
bzz:fix-rest-api-path-in-webapp
Closed

ZEPPELIN-31 fix REST API base path in web-app#30
bzz wants to merge 3 commits into
apache:masterfrom
bzz:fix-rest-api-path-in-webapp

Conversation

@bzz

@bzzbzz commented Apr 7, 2015

Copy link
Copy Markdown
Member

@bzz

bzz commented Apr 8, 2015

Copy link
Copy Markdown
MemberAuthor

Ready to merge

@Leemoonsoo

Copy link
Copy Markdown
Member

When i go to interpreter menu.

REST api call address was
http://localhost:8080/api/interpreter

but changed to
http://localhost:8080//api/interpreter

and getting 404

@bzz

bzz commented May 10, 2015

Copy link
Copy Markdown
MemberAuthor

@Leemoonsoo should be good now

@bzzbzz mentioned this pull request May 10, 2015
@bzzbzz closed this May 19, 2015
@bzz

bzz commented May 19, 2015

Copy link
Copy Markdown
MemberAuthor

Triggering CI

@bzzbzz reopened this May 19, 2015
@bzz

bzz commented May 19, 2015

Copy link
Copy Markdown
MemberAuthor

Ready to merge

@bzz

bzz commented Jun 8, 2015

Copy link
Copy Markdown
MemberAuthor

\cc @corneadoug@anthonycorbacho@minahlee for review

@corneadoug

Copy link
Copy Markdown
Contributor

+1

@anthonycorbacho

Copy link
Copy Markdown
Contributor

+1 as well
On Jun 11, 2015 11:18 PM, "CORNEAU Damien" notifications@github.com wrote:

+1


Reply to this email directly or view it on GitHub
#30 (comment)
.

@Leemoonsoo

Copy link
Copy Markdown
Member

Tested. +1

@bzz

bzz commented Jun 12, 2015

Copy link
Copy Markdown
MemberAuthor

Thanks, merging then!

@bzz
bzz deleted the fix-rest-api-path-in-webapp branch June 12, 2015 09:33
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.

4 participants

@bzz@Leemoonsoo@corneadoug@anthonycorbacho