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

Harden the service wall instance card #2908

Merged
merged 1 commit into from
Aug 30, 2018

Conversation

richard-cox
Copy link
Contributor

  • Ensure we only show the template, which requires serviceInstanceEntity, when we have it
  • Removed ServicesWallService provider, it's not used (ctor never called)
  • Removed/Sorted imports
  • Fixes I can not get service information #2814

- Ensure we only show the template, which requires serviceInstanceEntity, when we have it
- Removed ServicesWallService provider, it's not used (ctor never called)
- Removed/Sorted imports
@cfdreddbot
Copy link

Hey richard-cox!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA.

@codecov
Copy link

codecov bot commented Aug 24, 2018

Codecov Report

Merging #2908 into v2-master will decrease coverage by <.01%.
The diff coverage is 84.61%.

@@              Coverage Diff              @@
##           v2-master    #2908      +/-   ##
=============================================
- Coverage      71.35%   71.34%   -0.01%     
=============================================
  Files            604      604              
  Lines          25857    25856       -1     
  Branches        5855     5855              
=============================================
- Hits           18449    18448       -1     
  Misses          7408     7408

@irfanhabib
Copy link
Contributor

LGTM

@irfanhabib irfanhabib merged commit 06ccdf9 into v2-master Aug 30, 2018
@irfanhabib irfanhabib deleted the harden-service-instance-card branch August 30, 2018 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants