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

Changing how we check if services are present in order to show disabl… #217

Merged
merged 4 commits into from Oct 19, 2016

Conversation

jleonardw9
Copy link
Collaborator

…ed services.

@amcates
Copy link
Collaborator

amcates commented Oct 18, 2016

This is not 100% correct. You still need the old check as well. Otherwise they wouldn't be able to add PP SERVICES

On Oct 18, 2016, at 11:11 AM, Jason Leonard notifications@github.com wrote:

…ed services.

You can view, comment on, or merge this pull request online at:

#217

Commit Summary

Changing how we check if services are present in order to show disabled services.
File Changes

M app/controllers/protocols_controller.rb (2)
Patch Links:

https://github.com/sparc-request/sparc-fulfillment/pull/217.patch
https://github.com/sparc-request/sparc-fulfillment/pull/217.diff

You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@jleonardw9
Copy link
Collaborator Author

@amcates Ok, I forgot about that. It should be fixed now.

@jleonardw9 jleonardw9 merged commit 3008e09 into master Oct 19, 2016
@jleonardw9 jleonardw9 deleted the jl-hotfix-for-tab-display branch October 19, 2016 13:49
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

2 participants