-
Notifications
You must be signed in to change notification settings - Fork 48
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
Feature/#11 display processing errors #17
Conversation
…kie and js-errors reports
var service = { | ||
getMetadata: getMetadata, | ||
saveMetadata: saveMetadata | ||
}, | ||
requestParams = requestParametersService.get(), | ||
configParams = configService.getConfig(); | ||
getEndpointUrl = endpointConfiguration.getEndpointUrl(); |
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.
Could we rename getEndpointUrl
variable? To sth like
metadataUrl
endpoint
- ??
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.
Renamed.
'use strict'; | ||
angularAMD.factory('endpointConfiguration', function () { | ||
return { | ||
getEndpointUrl: function () { |
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.
As service is called endpointConfiguration~maybe method could be renmd to
getUrl` ?
What do you think?
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.
Yes, this is good idea - I renamed method to getEndpoint
and production
to getUrl
.
…ndable panel directive
…-performance-report fix for client side performance report
…atterns fix for accepting patterns on localhost:9000
…mment-popover fix for included comment popover
…or-filtering Status filter renamed and fix for filtering
Search filter for test renamed
Css for url tabs
For the source comparator there is a JavaScript error as Angular expects JSON object but artifactId is a HTML source. This should probably be investigated... |
fix for tabs buttons being scrolled away
This code is for #11 - processing errors are now displayed on report tabs.
I hereby agree to the terms of the AET Contributor License Agreement.