Skip to content

ADMINUI-2331 Support for viewing and update of service configurations#222

Open
andyYatskov wants to merge 1 commit into
TritonDataCenter:masterfrom
dan-luk:dev-2331
Open

ADMINUI-2331 Support for viewing and update of service configurations#222
andyYatskov wants to merge 1 commit into
TritonDataCenter:masterfrom
dan-luk:dev-2331

Conversation

@andyYatskov

Copy link
Copy Markdown
Contributor

No description provided.

@Arc311

Arc311 commented Aug 31, 2016

Copy link
Copy Markdown
Contributor

🆗

Comment thread lib/adminui.js
auth.requireRole('operators'),
resume,
bodyParser,
services.updateService);

Copy link
Copy Markdown
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

dan-luk commented Sep 1, 2016

Copy link
Copy Markdown
Contributor

🔢

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

dan-luk commented Sep 5, 2016

Copy link
Copy Markdown
Contributor

🔢

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

Copy link
Copy Markdown
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

dan-luk commented Sep 21, 2016

Copy link
Copy Markdown
Contributor

🔢

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