Skip to content

Conversation

@jaigak
Copy link
Contributor

@jaigak jaigak commented Jun 20, 2020

Implements and closes #606.

@ghost ghost added the ready for review Pull requests that are ready for review label Jun 20, 2020
@yaira2 yaira2 added changes requested Changes are needed for this pull request and removed ready for review Pull requests that are ready for review labels Jun 21, 2020
@jaigak
Copy link
Contributor Author

jaigak commented Jun 23, 2020

Please don't merge yet.

@jaigak jaigak changed the title Add the option to run an executable as an administrator Modify context menu options for executable and installer files Jun 23, 2020
@jaigak jaigak marked this pull request as draft June 23, 2020 14:37
@jaigak jaigak marked this pull request as ready for review June 24, 2020 02:59
@ghost ghost added the ready for review Pull requests that are ready for review label Jun 24, 2020
@jaigak
Copy link
Contributor Author

jaigak commented Jun 24, 2020

You can merge now. I've also added the option to run as another user. This option is hidden in File Explorer however I included it by default in Files UWP since it'd be helpful for users who have multiple user accounts.

@yaira2
Copy link
Member

yaira2 commented Jun 24, 2020

:shipit:

@yaira2 yaira2 requested a review from lukeblevins June 24, 2020 14:40
@yaira2 yaira2 removed the changes requested Changes are needed for this pull request label Jun 24, 2020
@lukeblevins lukeblevins self-requested a review June 25, 2020 16:53
@yaira2 yaira2 requested a review from tsvietOK June 25, 2020 17:01
@yaira2 yaira2 added ready to merge Pull requests that are approved and ready to merge and removed ready for review Pull requests that are ready for review labels Jun 25, 2020
@yaira2 yaira2 merged commit d2ac020 into files-community:master Jun 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to merge Pull requests that are approved and ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a run as admin option for .exe files

4 participants