Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Added fixes and tweaks #6

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

DariusHutchison
Copy link

No description provided.

Copy link
Owner

@freemansoft freemansoft left a comment

Choose a reason for hiding this comment

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

Noted a couple of typos and asked for a couple of lines that describe how to execute it using the external config file.


function defaultPrompt {
if ($null -eq $ConfigFile) {
$DefaultPrompt = Read-Host = "Would you like to load the default values to the config file? (Y/N)"
Copy link
Owner

Choose a reason for hiding this comment

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

Is this going to save defaults to a config file or load them from a config file. I'm not sure from the prompt.

$Script:StorageSpacesParams | ConvertTo-Json | Out-File $PSScriptRoot + "\TieredStorageSpace-Config.json"
}
elseif ($DefaultPrompt.ToUpper() -eq "N") {
Read-Host "Ether pass the config file to the script or use default values. Exiting..."
Copy link
Owner

Choose a reason for hiding this comment

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

"Ether" should be "Either"

@@ -1,8 +1,11 @@
# Create Storage Spaces in Windows 10

<!--TODO: Update README documrntaion to match new scripts usage. /--->
Copy link
Owner

Choose a reason for hiding this comment

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

documrntation should be "documentation".


![Simple](./images_folder/simple.png) ![Mirrored](./images_folder/mirror-simple.png) ![Mirrored and Striped](./images_folder/mirror-stripe.png)

# Scripts
## new-storage-space.ps1
## New-TieredStorageSpace.ps1
Creates a tiered storage pool and allocates all the disk space to a single drive
Copy link
Owner

Choose a reason for hiding this comment

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

Can we add a line here or at the top that tells us how to run the script with the external configuration file?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants