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

add rxjs dependencies for test #461

Merged

PatrickJS merged 1 commit into PatrickJS:master from ktong:master

Mar 28, 2016

Contributor

ktong commented Mar 22, 2016

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Bug fix: test failed when service use map from rxjs

What is the current behavior? (You can also link to an open issue here)

Failed: undefined is not a constructor

What is the new behavior (if this is a feature change)?

Test passed

Other information:


          add rxjs dependencies for test

c0f817d

Owner

PatrickJS commented Mar 28, 2016

good catch

PatrickJS merged commit 233fa09 into PatrickJS:master

MaxMood96 mentioned this pull request

[Snyk] Fix for 1 vulnerabilities MaxMood96/angular-starter#29

Open

MaxMood96 mentioned this pull request

[Snyk] Fix for 1 vulnerabilities MaxMood96/angular-starter#34

Open

MaxMood96 mentioned this pull request

[Snyk] Fix for 2 vulnerabilities MaxMood96/angular-starter#56

Open

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet