Skip to content

Conversation

@andyYatskov
Copy link
Contributor

No description provided.

@Arc311
Copy link
Contributor

Arc311 commented Aug 31, 2016

🆗

auth.requireRole('operators'),
resume,
bodyParser,
services.updateService);
Copy link
Contributor

Choose a reason for hiding this comment

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

remove two lines below

@dan-luk
Copy link
Contributor

dan-luk commented Sep 1, 2016

🔢

@andyYatskov andyYatskov force-pushed the dev-2331 branch 4 times, most recently from caebda7 to 3474426 Compare September 5, 2016 10:44
@dan-luk
Copy link
Contributor

dan-luk commented Sep 5, 2016

🔢

</h1>
</div>
{this.state.error && <ErrorAlert error={this.state.error} />}
<section className="params">
Copy link
Contributor

Choose a reason for hiding this comment

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

Angela Fong
8:55 AM

This entire section (L75:L114) is actually not required to be displayed for a service: 1) Customer metadata are not applicable to services, 2) Tags are view-only and aren't supposed to be something user has control over.

dan-luk
dan-luk previously approved these changes Sep 21, 2016
@dan-luk
Copy link
Contributor

dan-luk commented Sep 21, 2016

🔢

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.

3 participants