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.

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

Add support for skipLoadingService configuration parameter #64

Conversation

szabyg
Copy link

@szabyg szabyg commented Aug 17, 2016

Fixes #63

@szabyg szabyg force-pushed the errorHttpInterceptor-with-skipLoadingService-suport branch from c99f95b to cd4df50 Compare August 17, 2016 15:02
@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 53.196% when pulling cd4df50 on Eurofunk:errorHttpInterceptor-with-skipLoadingService-suport into cce80eb on Catalysts:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 53.196% when pulling cd4df50 on Eurofunk:errorHttpInterceptor-with-skipLoadingService-suport into cce80eb on Catalysts:master.

return config;
},
'requestError': function (rejection) {
loadingService.stop();
if (!toBeIgnored(rejection, rejection.config)) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

toBeIgnored only takes one parameter, I guess you only meant to pass rejection.config here?

@szabyg szabyg force-pushed the errorHttpInterceptor-with-skipLoadingService-suport branch from cd4df50 to 7f89035 Compare August 25, 2016 05:59
@coveralls
Copy link

Coverage Status

Coverage increased (+0.2%) to 53.196% when pulling 7f89035 on Eurofunk:errorHttpInterceptor-with-skipLoadingService-suport into cce80eb on Catalysts:master.

@HaraldVogl HaraldVogl merged commit c12a5bd into Catalysts:master Aug 25, 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.

None yet

4 participants