Skip to content

Add conversion to ESD - #4

Merged
CodingWonders merged 2 commits into
CodingWonders:microwin-202506from
Real-MullaC:microwin-202506
Aug 1, 2025
Merged

Add conversion to ESD#4
CodingWonders merged 2 commits into
CodingWonders:microwin-202506from
Real-MullaC:microwin-202506

Conversation

@Real-MullaC

@Real-MullaC Real-MullaC commented Jul 31, 2025

Copy link
Copy Markdown

Type of Change

  • New feature
  • Bug fix
  • Documentation update
  • Refactoring
  • Hotfix
  • Security patch
  • UI/UX improvement

Description

Testing

Impact

Issue related to PR

Additional Information

Checklist

  • My code adheres to the coding and style guidelines of the project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no errors/warnings/merge conflicts.

@Real-MullaC
Real-MullaC marked this pull request as draft July 31, 2025 23:17
@Real-MullaC

Copy link
Copy Markdown
Author
image

Added quotes to the file paths. Put all the arguments in 1 string (as that also works fine)

@CodingWonders CodingWonders left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

I made some changes to the DISM process start to make sure it doesn't fail when the file paths contain spaces. You can escape quotes by typing the following:

`"path`"

The list of arguments has also been merged into 1 string since that also works fine.

Anyway. This is a good PR

@CodingWonders
CodingWonders marked this pull request as ready for review August 1, 2025 05:44
@CodingWonders

CodingWonders commented Aug 1, 2025

Copy link
Copy Markdown
Owner

I also recommend that you tick the checkboxes in the PR body. I think you made the templates; why not use them?

@CodingWonders
CodingWonders merged commit 4cccbb6 into CodingWonders:microwin-202506 Aug 1, 2025
CodingWonders added a commit that referenced this pull request Aug 5, 2025
* Disable "Microsoft account" notification source

* [WinPE] Remove driver addition block for WinPE

WE DO NOT TOUCH THE DRIVERS IN WinPE. There are reports of people getting "Install driver to show hardware" screens all over this repository, and on Discord; and the less drivers we touch in WinPE, the better.

Drivers can still be added to Preinstallation Environments in the following ways:

- Using the driver installation screens
- Firing up "drvload.exe <driver>" in cmd

* [Fix] Added fallback for DISM export command

This is a port of the fix in ChrisTitusTech#3305

* [Cleanup] Removed some comments that no longer make sense

* [Fix] Same DISM export image fallback fix

* Merge branch 'main' into microwin-202506

* [Fix] Improve UI consistency for instructions

Fixes ChrisTitusTech#3394

* Merge branch 'main' into microwin-202506

* [Unattended answer file] Remove it from drive root

The answer file, on the drive root, is not necessary for us to apply it. In fact, it's not even used there

* Merge branch 'main' into microwin-202506

* [MicroWin] June-August 2025 Update (#3) -- Contributions from Callum

* Allow people without compatible hardware or a USB to use MicroWin.

* Update functions/microwin/Microwin-NewUnattend.ps1

Co-authored-by: CodingWonders <101426328+CodingWonders@users.noreply.github.com>

* Update Invoke-Microwin.ps1

* Update Microwin-NewUnattend.ps1

* Update Microwin-NewUnattend.ps1

* Add error pop up if ISO Creation fails.

Issue 2653

* Add Disable WPBT Execution to MicroWin.

* Update functions/microwin/Invoke-Microwin.ps1

Co-authored-by: CodingWonders <101426328+CodingWonders@users.noreply.github.com>

* modified:   functions/microwin/Invoke-Microwin.ps1
	modified:   xaml/inputXML.xaml

---------

Co-authored-by: CodingWonders <101426328+CodingWonders@users.noreply.github.com>

* Add conversion to ESD (#4)

* Add conversion to ESD

Issue - ChrisTitusTech#3450

* Update Invoke-Microwin.ps1

Added quotes to the file paths. Put all the arguments in 1 string (as that also works fine)

---------

Co-authored-by: CodingWonders <101426328+CodingWonders@users.noreply.github.com>

* Update MicroWin contributor list

* Merge branch 'main' into microwin-202506

* Merge branch 'main' into microwin-202506

* [MicroWin] Add automatic configuration settings

Originally implemented in ChrisTitusTech#2618. Adapted to follow the new file structure. And it works.

Though there are issues that will be detailed very soon

* [Fix] Fixed typos, updated descriptions

* Re-add WinPE driver addition

We're not yet sure if that is the actual problem of missing storage controllers. Logs can tell us more about this. Maybe for a future PR?

* [Fix/WPBT] Add spaces to reg key path

Avoid REG failure

* [Fix/XAML] Fix word wrapping issue for checkboxes
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.

2 participants