Skip to content

fixing up an issue where the radar crashes if an outer layer is filled i...#9

Open
hasTimc wants to merge 1 commit intobdargan:masterfrom
hasTimc:master
Open

fixing up an issue where the radar crashes if an outer layer is filled i...#9
hasTimc wants to merge 1 commit intobdargan:masterfrom
hasTimc:master

Conversation

@hasTimc
Copy link

@hasTimc hasTimc commented Mar 10, 2015

...n without all inner layers filled in. i.e. if you have an item in hold and 3 in adopt with nothing in trial or assess the picture will not display without this fix.

…d in without all inner layers filled in. i.e. if you have an item in hold and 3 in adopt with nothing in trial or assess the picture will not display without this fix.
@NetAppBlueDevil
Copy link

I think this solves the same problem I was seeing. I did slightly differently since I didn't understand if the key thing was important. I just got a dataIndex (below) to use for the points, but I think it solves the same issue the for loop setup in this change addresses

My tweak
for (var stageIdx in _(itemsByStage).keys()) {
dataIndex = _(itemsByStage).keys()[stageIdx]

amind-depvos added a commit to amind-depvos/techradar that referenced this pull request Jul 8, 2016
See pull request for more info bdargan#9
amind-depvos added a commit to mjukutv/techradar-semantics that referenced this pull request Jul 8, 2016
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

Comments