Skip to content
This repository was archived by the owner on Sep 17, 2021. It is now read-only.

Conversation

@kalpatel01
Copy link
Contributor

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 54.416% when pulling ce4a430 on Bridgewater:9582_JustifiedReport into 86c4a7e on Netflix:develop.

Patrick Kelley added 2 commits February 22, 2017 10:56
Adding an extra `/-` in the justified url to account for AccountType.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 54.416% when pulling 21e5e71 on Bridgewater:9582_JustifiedReport into 86c4a7e on Netflix:develop.

<li><a href="#/issues/-/redshift/-/-/-/-/True/-/1/25">Redshift Issues</a></li>
<li class="divider"></li>
<li><a href="#/justified/-/-/-/-/-/-/-/1/25">Justified Issues</a></li>
<li><a href="#/justified/-/-/-/-/-/-/-/-/1/25">Justified Issues</a></li>

Choose a reason for hiding this comment

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

Whoops, good catch! I think we must have prepped this at the same time that @supertom made his changes for the account type :)

}),
'justified': ngRoute(
path: '/justified/:regions/:technologies/:accounts/:names/:arns/:active/:searchconfig/:page/:count',
path: '/justified/:regions/:technologies/:accounts/:accounttypes/:names/:arns/:active/:searchconfig/:page/:count',

Choose a reason for hiding this comment

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

Ditto my last comment....I think we must have prepped this at the same time that @supertom made his changes for the account type :)

@scriptsrc scriptsrc merged commit 83e572a into Netflix:develop Feb 22, 2017
@coveralls
Copy link

Coverage Status

Coverage remained the same at 54.416% when pulling 21e5e71 on Bridgewater:9582_JustifiedReport into 86c4a7e on Netflix:develop.

This was referenced Apr 14, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants