Skip to content

PowerPlan: Add a read-only property IsActive #171

Closed
#194
#193
Closed
PowerPlan: Add a read-only property IsActive#171
#194
#193
Assignees
PlagueHO
Labels
enhancementThe issue is an enhancement request.good first issueThe issue should be easier to fix and can be taken up by a beginner to learn to contribute on GitHub

Description

@johlju
johlju
opened on Jul 20, 2018

Details of the scenario you tried and the problem that is occurring:
Suggest we add a read-only property IsActive to the resource PowerPlan, and set that to $plan.isActive instead of setting the $Name to $null in Get-TargetResource if the plan is not the active plan.

https://github.com/PowerShell/ComputerManagementDsc/blob/c0a60754c2ec537f1668e4a3cf6ca7423a8c9a2c/Modules/ComputerManagementDsc/DSCResources/MSFT_PowerPlan/MSFT_PowerPlan.psm1#L67-L76

The DSC configuration that is using the resource (as detailed as possible):
Any configuration that hasn't the power plan as the active power plan.

Version of the Operating System and PowerShell the DSC Target Node is running:
n/a

Version of the DSC module you're using, or 'dev' if you're using current dev branch:
Dev

Activity

  1. johlju
    changed the title [-]PowerPlan: Add a red-only property IsActive[/-] [+]PowerPlan: Add a read-only property IsActive[/+] on Jul 20, 2018
  2. PlagueHO
    added
    enhancementThe issue is an enhancement request.
    help wantedThe issue is up for grabs for anyone in the community.
    good first issueThe issue should be easier to fix and can be taken up by a beginner to learn to contribute on GitHub
    on Jul 24, 2018
  3. PlagueHO
    closed this as completedin #193on Oct 11, 2018
  4. PlagueHO
    added a commit that references this issue on Oct 11, 2018

    Merge pull request #193 from TraGicCode/GH-171-add-is-active-read-onl…

    99c25ff
  5. johlju
    removed
    help wantedThe issue is up for grabs for anyone in the community.
    on Oct 11, 2018
  6. johlju
    assigned
    PlagueHO
    on Oct 11, 2018
  7. PlagueHO
    added a commit that references this issue on Oct 11, 2018

    Merge pull request #194 from PlagueHO/Issue-171-Fix

    d7679dc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

enhancementThe issue is an enhancement request.good first issueThe issue should be easier to fix and can be taken up by a beginner to learn to contribute on GitHub

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions