Skip to content
This repository has been archived by the owner on Jul 6, 2020. It is now read-only.

fix: Fix Endpoint Service not being injected in self object #117

Merged
merged 4 commits into from
May 9, 2019

Conversation

mtahafarooq
Copy link
Contributor

Fixes # 116

Changes proposed in this pull request:

  • Added the endpointService inside the constructor of the signup component which fixed the endpointService not being found in the self object.

@mtahafarooq
Copy link
Contributor Author

This PR fixes 116 issue.

@codecov-io
Copy link

codecov-io commented Apr 3, 2019

Codecov Report

Merging #117 into master will increase coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #117      +/-   ##
==========================================
+ Coverage   53.39%   53.43%   +0.03%     
==========================================
  Files          55       55              
  Lines        2358     2360       +2     
  Branches      253      253              
==========================================
+ Hits         1259     1261       +2     
  Misses       1025     1025              
  Partials       74       74
Impacted Files Coverage Δ
src/app/components/auth/signup/signup.component.ts 75.75% <100%> (+1.56%) ⬆️
Impacted Files Coverage Δ
src/app/components/auth/signup/signup.component.ts 75.75% <100%> (+1.56%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9e9d5b9...2e72141. Read the comment docs.

Copy link
Member

@Shekharrajak Shekharrajak left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@lunayach
Copy link
Member

lunayach commented Apr 3, 2019

Nice find. Shall we merge this @Shekharrajak @RishabhJain2018?

@Shekharrajak
Copy link
Member

Yes! I believe it can be merged, after another approval.

@Shekharrajak Shekharrajak merged commit 64ac4eb into Cloud-CV:master May 9, 2019
sanketbansal pushed a commit to sanketbansal/EvalAI-ngx that referenced this pull request May 18, 2019
…#117)

* Endpoint Service was not being injected in self object

* Add Endpoint service in test file providers
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants