Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Two separate bugfixes. #1120

Merged
merged 2 commits into from
May 27, 2016
Merged

Two separate bugfixes. #1120

merged 2 commits into from
May 27, 2016

Conversation

NegatioN
Copy link
Contributor

  1. Bugfix goToTrace not triggering on other pages than default, creating a malformed url.
  2. webpack devproxy only targeted default-page locally, and got resources from upstream api if you attached an external dev-server. The external zipkin-server can now be configured with an environment-variable.

1. Bugfix goToTrace not triggering on other pages than default, creating a malformed url.
2. webpack devproxy only targeted default-page locally, and got resources from upstream api if you attached an external dev-server. The external zipkin-server can now be configured with an environment-variable.
@codefromthecrypt
Copy link
Member

thanks!

mind putting in the README how to connect to an existing zipkin server (ex via docker or some other specified ip)?

@NegatioN
Copy link
Contributor Author

I'm not sure if this is exactly what you were looking for. Please give me a shout if it wasn't. :)

@eirslett
Copy link
Contributor

:shipit:

@codefromthecrypt codefromthecrypt merged commit 8c644b5 into openzipkin:master May 27, 2016
@codefromthecrypt
Copy link
Member

Thanks!

@virtuald
Copy link
Contributor

Heh. Ran into this today when I tried to modify the UI, if only if I had seen it before my flight...

@codefromthecrypt
Copy link
Member

codefromthecrypt commented May 29, 2016 via email

@eirslett
Copy link
Contributor

Flight attendants should remind people to git pull as part of the preflight instructions...

@codefromthecrypt
Copy link
Member

codefromthecrypt commented May 29, 2016 via email

codefromthecrypt pushed a commit that referenced this pull request Jun 3, 2016
* Two separate bugfixes.
1. Bugfix goToTrace not triggering on other pages than default, creating a malformed url.
2. webpack devproxy only targeted default-page locally, and got resources from upstream api if you attached an external dev-server. The external zipkin-server can now be configured with an environment-variable.

* update readme with proxy backend-info
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug ui Zipkin UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants