Skip to content

Conversation

@JPrevost
Copy link
Member

@JPrevost JPrevost commented Mar 8, 2017

EDS upstream fix to our sso access seems to have forced sso into our
EDS API result links. This is a hacky way to address that while we
consider better options

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling d603b911941413b8dd6bdd6cc07c661b148599b9 on 248_fix_guest_access_fix into 15bdd10 on master.

@gravesm gravesm temporarily deployed to mit-bento-staging-pr-146 March 8, 2017 21:26 Inactive
@gravesm gravesm temporarily deployed to mit-bento-staging-pr-146 March 8, 2017 21:28 Inactive
EDS upstream fix to our sso access seems to have forced sso into our
EDS API result links. This is a hacky way to address that while we
consider better options
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 51c8a39 on 248_fix_guest_access_fix into 686af24 on master.

Copy link
Contributor

@frrrances frrrances left a comment

Choose a reason for hiding this comment

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

super. 👍

@JPrevost JPrevost merged commit b8c5096 into master Mar 9, 2017
@JPrevost JPrevost deleted the 248_fix_guest_access_fix branch March 9, 2017 14:39
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.

5 participants