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

Admin elevation when runs as regular user #2565

Closed
wants to merge 1 commit into from

Conversation

ChrisTitusTech
Copy link
Owner

@ChrisTitusTech ChrisTitusTech commented Aug 14, 2024

Pull Request

Title

Elevate Prompt to Admin so script can run

Type of Change

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

Description

Normally the script would exit if run as a normal user.

Testing

Testing not done.

Impact

This should check to see if Windows Terminal, Powershell 7, or Powershell is installed. It will adjust the run command based on the users system

Additional Information

Currently another version done different in PR #2531 and wanted a simple approach for non-elevated runs

Checklist

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

@ChrisTitusTech ChrisTitusTech mentioned this pull request Aug 14, 2024
6 tasks
@MyDrift-user
Copy link
Contributor

There is one Issue with this version of relaunching as admin. If someone is running the dev winutil release or running his just compiled local version of winutil for testing, it will use the latest full release. It would be great if we could pass $myinvocation to the newly opened shell to run the correct script.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-changelog Skip Change Logs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants