From 534d81c26a2c7027f4b4a172c1a252d027a119f6 Mon Sep 17 00:00:00 2001 From: Joe Farro Date: Wed, 25 Oct 2017 15:15:18 -0400 Subject: [PATCH 1/2] Add changelog (#104) * Add a changelog, dates to 2017-08-23 Signed-off-by: Joe Farro --- CHANGELOG.md | 62 ++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 62 insertions(+) create mode 100644 CHANGELOG.md diff --git a/CHANGELOG.md b/CHANGELOG.md new file mode 100644 index 0000000000..e43a65b505 --- /dev/null +++ b/CHANGELOG.md @@ -0,0 +1,62 @@ +# Changes + + +### [#97](https://github.com/jaegertracing/jaeger-ui/pull/97) Change to Apache license v.2 and add DCO / CONTRIBUTING.md + + +### [#93](https://github.com/jaegertracing/jaeger-ui/pull/93) Keyboard shortcuts and minimap UX + +- Fix [#89](https://github.com/uber/jaeger-ui/issues/89) - [trace view] Drag and release on timeline header row zooms into respective range +- Fix [#23](https://github.com/uber/jaeger-ui/issues/23) - [trace view] Navigate and zoom via minimap +- Fix [#22](https://github.com/uber/jaeger-ui/issues/22) - [trace view] Pan and zoom via keyboard shortcuts + + +### [#84](https://github.com/jaegertracing/jaeger-ui/pull/84) Improve search dropdowns + +- Fix [#79](https://github.com/uber/jaeger-ui/issues/79) - Sort services and operations operations (case insensitive) +- Fix [#31](https://github.com/uber/jaeger-ui/issues/31) - Filter options based on contains instead of starts with +- Fix [#30](https://github.com/uber/jaeger-ui/issues/30) - Filter options based on case insensitive match + + +### [#78](https://github.com/jaegertracing/jaeger-ui/pull/78) Custom menu via /api/config with project links as defaults + +- Fix [#44](https://github.com/uber/jaeger-ui/issues/44) - Add configurable, persistent links to the header +- **Support for this is WIP in query service** + + +### [#81](https://github.com/jaegertracing/jaeger-ui/pull/81) Fix Google Analytics tracking + + +### [#77](https://github.com/jaegertracing/jaeger-ui/pull/77) Fix trace mini-map blurry when < 60 spans + + +### [#74](https://github.com/jaegertracing/jaeger-ui/pull/74) Make left column adjustable in trace detail + + +### [#71](https://github.com/jaegertracing/jaeger-ui/pull/71) [trave view] Mouseover expands truncated text to full length in left column + + +### [#68](https://github.com/jaegertracing/jaeger-ui/pull/68) Virtualized scrolling for trace detail view + +- Performance improved for initial loading, expanding span details, text search and scrolling + + +### [#53](https://github.com/jaegertracing/jaeger-ui/pull/53) Refactor trace detail + +- Partial fix for [#42](https://github.com/uber/jaeger-ui/issues/42) - Support URL prefix via homepage in package.json +- Scatterplot dots are sized based on number of spans +- Scatterplot dots mouseover shows trace name +- Clicking span detail left column collapses detail +- Clicking anywhere left of parent span name toggles children visibility +- Clip or hide span bars when zoomed in (instead of flush left) +- Label on span bars no longer off-screen +- Full width of the header is clickable for tags, process, and logs headers (instead of header text, only) +- Horizontal scrolling for wide content (e.g. long log values) (Fix [#58](https://github.com/uber/jaeger-ui/issues/58)) +- Tall content scrolls via entire table instead of single table cell +- Fix [#55](https://github.com/uber/jaeger-ui/issues/55) - Some tags were not being rendered due to clashing keys (observed in a log message) +- Fix [jaegertracing/jaeger#326](https://github.com/jaegertracing/jaeger/issues/326) - extraneous scrollbars in trace views +- Ticks in span graph made to match trace detail (in number and formatting) +- Fix [#49](https://github.com/uber/jaeger-ui/issues/42) - Span position in graph doesn't not match its position in the detail + + +### [Changes from before 2017-08-23 are not logged here](https://www.youtube.com/watch?v=NoAzpa1x7jU&feature=youtu.be&t=107) From 06f5c65b9f2e86b9b6016b2df71594c5fd4f9108 Mon Sep 17 00:00:00 2001 From: Joe Farro Date: Wed, 25 Oct 2017 16:26:32 -0400 Subject: [PATCH 2/2] Fix #105 test error, fetch requires absolute urls Signed-off-by: Joe Farro --- src/actions/jaeger-api.test.js | 12 ++++++++++-- src/middlewares/index.test.js | 12 +++++++++++- 2 files changed, 21 insertions(+), 3 deletions(-) diff --git a/src/actions/jaeger-api.test.js b/src/actions/jaeger-api.test.js index d26c677b50..aad70b1044 100644 --- a/src/actions/jaeger-api.test.js +++ b/src/actions/jaeger-api.test.js @@ -12,12 +12,20 @@ // See the License for the specific language governing permissions and // limitations under the License. +/* eslint-disable import/first */ +jest.mock('node-fetch', () => () => + Promise.resolve({ + status: 200, + data: () => Promise.resolve({ data: null }), + json: () => Promise.resolve({ data: null }), + }) +); + import sinon from 'sinon'; import isPromise from 'is-promise'; -import JaegerAPI from '../api/jaeger'; - import * as jaegerApiActions from './jaeger-api'; +import JaegerAPI from '../api/jaeger'; it('@JAEGER_API/FETCH_TRACE should fetch the trace by id', () => { const api = JaegerAPI; diff --git a/src/middlewares/index.test.js b/src/middlewares/index.test.js index 62b66fd596..233d0e0b09 100644 --- a/src/middlewares/index.test.js +++ b/src/middlewares/index.test.js @@ -12,8 +12,18 @@ // See the License for the specific language governing permissions and // limitations under the License. -import sinon from 'sinon'; +/* eslint-disable import/first */ +jest.mock('node-fetch', () => () => + Promise.resolve({ + status: 200, + data: () => Promise.resolve({ data: null }), + json: () => Promise.resolve({ data: null }), + }) +); + import { change } from 'redux-form'; +import sinon from 'sinon'; + import * as jaegerMiddlewares from './index'; import { fetchServiceOperations } from '../actions/jaeger-api';