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

Fixes #9885: In group page, remove reference to CFEngine in generated classes #1419

Conversation

VinceMacBuche
Copy link
Member

@@ -177,7 +177,7 @@ class NodeGroupForm(
"rudderid" -> <div><b class="threeCol">Rudder ID: </b>{nodeGroup.id.value}</div>,
"cfeclasses" -> <div>
<a href="#" onclick={s"$$('#cfe-${nodeGroup.id.value}').toggle(300); return false;"}>
<b class="threeCol">Display CFEngine classes</b>
<b class="threeCol">Display classes generated for this group</b>
Copy link
Member

Choose a reason for hiding this comment

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

I think that's a bit long. How about "Display conditional classes"

Copy link
Member Author

Choose a reason for hiding this comment

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

Love to have a shorter / lighter thing.

Why not for "conditional". (but maybe people will not understand where it can be used)

My idea; "display agent classes", what do you think of it ?

Copy link
Member

Choose a reason for hiding this comment

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

I like it! Perfect.

@VinceMacBuche
Copy link
Member Author

Commit modified

@VinceMacBuche VinceMacBuche force-pushed the bug_9885/in_group_page_remove_reference_to_cfengine_in_generated_classes branch from 10de26a to c669cd0 Compare January 2, 2017 16:10
@Normation-Quality-Assistant
Copy link
Contributor

This PR is not mergeable to upper versions.
Since it is "Ready for merge" you must merge it by yourself using the following command:
rudder-dev merge https://github.com/Normation/rudder/pull/1419
-- Your faithful QA

@VinceMacBuche
Copy link
Member Author

OK, merging this PR

@VinceMacBuche VinceMacBuche merged commit c669cd0 into Normation:branches/rudder/3.2 Jan 3, 2017
@VinceMacBuche
Copy link
Member Author

OK, merging this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants