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.

Sign up for GitHub

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

Jump to bottom

WINDUP-880 expandMemory() causes JavaScript timeouts on big apps. #806

Merged
merged 4 commits into from Jan 15, 2016
Merged

WINDUP-880 expandMemory() causes JavaScript timeouts on big apps. #806

merged 4 commits into from Jan 15, 2016

Conversation

OndraZizka
Copy link
Contributor

Follows PR #779.

@OndraZizka
WINDUP-665 Tags tree impl
43b80c8
@OndraZizka
WINDUP-821 Replace the pie chart with a bar chart WIP
848b19c 
drop unused CSS
WINDUP-820 TreeView of projects
WINDUP-665 - Tag data; add human friendly title to tags (as a supplement to bare id-like names)
Tweaked overview loading logic to remove some imports that aren't compatible with htmlunit's javascript interpretter
@OndraZizka
Making the loading faster - WIP
7ac2bc9
@windupgithubbot1
Copy link

Refer to this link for build results (access rights to CI server needed):
http://wonka.mw.lab.eng.bos.redhat.com/jenkins//job/windup-core-PR-dependents-check/738/

@windupgithubbot1
Copy link

Refer to this link for build results (access rights to CI server needed):
http://wonka.mw.lab.eng.bos.redhat.com/jenkins//job/windup-core-PR-dependents-check/742/

@windupgithubbot1
Copy link

Refer to this link for build results (access rights to CI server needed):
http://wonka.mw.lab.eng.bos.redhat.com/jenkins//job/windup-core-PR-dependents-check/744/

@windupgithubbot1
Copy link

Refer to this link for build results (access rights to CI server needed):
http://wonka.mw.lab.eng.bos.redhat.com/jenkins//job/windup-core-PR-dependents-check/745/

@windupgithubbot1
Copy link

Refer to this link for build results (access rights to CI server needed):
http://wonka.mw.lab.eng.bos.redhat.com/jenkins//job/windup-pr-builder/1373/

Failed Tests: 1

windup-pr-builder/org.jboss.windup.tests:windup-tests: 1

@OndraZizka
Copy link
Contributor Author

Fixed the last test

@OndraZizka OndraZizka mentioned this pull request Jan 14, 2016
Closed
@OndraZizka OndraZizka closed this Jan 14, 2016
@OndraZizka OndraZizka reopened this Jan 14, 2016
@windupgithubbot1
Copy link

Refer to this link for build results (access rights to CI server needed):
http://wonka.mw.lab.eng.bos.redhat.com/jenkins//job/windup-core-PR-dependents-check/749/

@windupgithubbot1
Copy link

Refer to this link for build results (access rights to CI server needed):
http://wonka.mw.lab.eng.bos.redhat.com/jenkins//job/windup-pr-builder/1377/

Failed Tests: 1

windup-pr-builder/org.jboss.windup.tests:windup-tests: 1

@mareknovotny
Copy link
Contributor

I see 2 issues there
empty or not rendered graph in panel for Java Issues by Package on Report Index
screenshot from 2016-01-15 13-38-02
and second issue on Application detail where are shown both links for Collapse All/Expand All
screenshot from 2016-01-15 13-34-12

@mareknovotny
Copy link
Contributor

And there are not working tag label popups which on master worked before I tried this PR.
see title attribute is missing in your PR https://github.com/windup/windup/blame/master/rules-java/api/src/main/resources/reports/templates/java_application.ftl#L8
pointing to my own comment at OndraZizka@848b19c#commitcomment-15473883

@jsight jsight merged commit 2470eb5 into windup:master Jan 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@OndraZizka @windupgithubbot1 @mareknovotny @jsight