-
Notifications
You must be signed in to change notification settings - Fork 304
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
FISH-6483 add several other js require in admingui #5913
Conversation
Jenkins test, please |
Jenkins test please |
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 got this:
uploadFrame.jsf:48 Uncaught TypeError: Cannot read properties of undefined (reading 'value')
at HTMLInputElement.onclick (uploadFrame.jsf:48:63)
When I did a deploy.
The message was showed in Chrome Web Tools Console
Jenkins test, please |
@arieki |
I believe you want to mention @luiseufrasio instead |
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.
LGTM! It was fixed
Jenkins test please |
Create fake object with id and value, so it passes various checks. Later, when module is loaded, the checks are done correctly.
de15808
to
6b04d53
Compare
Jenkins test please |
Description
Added require for function calls on table, so they succeed.
Currently, there is no (known) JS warning in the log while working with admin gui.
There is only 404 for a js file (will be consecutive PR).
Important Info
Testing
Testing Performed
Going through admin gui (e.g. select list of JDBC resources, go to detail, ping, deploy application, etc.)
Testing Environment
Linux, OpenJDK 11
Notes for Reviewers
Open JS debugger while going through admin gui and watch uncatched exceptions.