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
HPCC-18601 Re-evaluate package maps widget in 7.0 #10958
Conversation
https://track.hpccsystems.com/browse/HPCC-18601 |
@GordonSmith Please review |
@GordonSmith rebased. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks ok - just some commented code to remove.
// resize: function (args) { | ||
// this.inherited(arguments); | ||
// this.borderContainer.resize(); | ||
// }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove commented code.
https://track.hpccsystems.com/browse/HPCC-18601 |
@GordonSmith Removed comments and rebased |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suspect this will fail linting due to duplicate vars.
var objStore = ESPPackageProcess.CreatePackageMapQueryObjectStore(); | ||
this.packagesGrid.setStore(objStore); | ||
} | ||
}, this.id + "PackagesGrid"); | ||
|
||
var context = this; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Already declared.
if (active === true) { | ||
return "A"; | ||
initPackagesGrid: function () { | ||
var context = this; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Already declared.
@GordonSmith Updated. |
@miguelvazq failing smoketest build. |
@GordonSmith I am not seeing where the smoketest would fail? Is this suspect? |
@miguelvazq if you replace |
Improving some of the code in PackageMapQueryWidget.js to take advantage of the ECLWatch framework. U/I was not loading in 7.0.0 due to some changes in dependencies - that has been fixed. Also, improve overall usability and performance. Signed-off by: Miguel Vazquez <miguel.vazquez@lexisnexis.com>
15c9a9d
to
58564ce
Compare
Automated Smoketest: ✅ Rebuild: success Unit tests result:
Regression test result:
HPCC Stop: OK |
@GordonSmith Should be good now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
Improving some of the code in PackageMapQueryWidget.js to take advantage
of the ECLWatch framework. U/I was not loading in 7.0.0 due
to some changes in dependencies - that has been fixed. Also, improve overall usability and
performance.
Signed-off by: Miguel Vazquez miguel.vazquez@lexisnexis.com
Type of change:
Checklist:
Testing: