Skip to content
This repository has been archived by the owner on Jul 12, 2022. It is now read-only.

[FIX] Windows Installer no longer requires admin privilege #291

Merged
merged 20 commits into from
Jul 17, 2020

Conversation

douglasvinter
Copy link

- What I did

  • Created a customized wix template with "perUser" installation (the default uses "perMachine") with limited privileges
  • Removed shortcut creation (Doesn't make sense to create a shortcut for a CLI tool without a embedded bash)
  • Removed the ico.ico (No shortcut, no icon needed)
  • Added regedit HKCU, enforcing no administrator required (HKEY_CURRENT_USER->Software->ZupIT->rit)
  • Added the necessary parameters to windows circleci script for template generation

- How to verify it

  • Build a new version and install in a user without Admin privileges (Worked with local build, need to test with automated build)

- Description for the changelog

  • Windows Installer no longer requires admin privilege

@ernelio
Copy link
Contributor

ernelio commented Jul 17, 2020

/merge qa

@ritchie-bot
Copy link
Contributor

ritchie-bot bot commented Jul 17, 2020

👌 Merged branch master into qa

@ernelio
Copy link
Contributor

ernelio commented Jul 17, 2020

/merge qa

@ritchie-bot
Copy link
Contributor

ritchie-bot bot commented Jul 17, 2020

👌 Merged branch master into qa

@ernelio
Copy link
Contributor

ernelio commented Jul 17, 2020

/merge qa

@ritchie-bot
Copy link
Contributor

ritchie-bot bot commented Jul 17, 2020

👌 Merged branch master into qa

@ernelio
Copy link
Contributor

ernelio commented Jul 17, 2020

/merge qa

@ritchie-bot
Copy link
Contributor

ritchie-bot bot commented Jul 17, 2020

👌 Merged branch master into qa

@ernelio
Copy link
Contributor

ernelio commented Jul 17, 2020

/merge qa

@ritchie-bot
Copy link
Contributor

ritchie-bot bot commented Jul 17, 2020

👌 Merged branch master into qa

@ernelio
Copy link
Contributor

ernelio commented Jul 17, 2020

/merge qa

@ritchie-bot
Copy link
Contributor

ritchie-bot bot commented Jul 17, 2020

👌 Merged branch master into qa

@ernelio
Copy link
Contributor

ernelio commented Jul 17, 2020

/merge qa

@ritchie-bot
Copy link
Contributor

ritchie-bot bot commented Jul 17, 2020

👌 Merged branch master into qa

@ernelio
Copy link
Contributor

ernelio commented Jul 17, 2020

/merge qa

@ritchie-bot
Copy link
Contributor

ritchie-bot bot commented Jul 17, 2020

👌 Merged branch master into qa

@ernelio
Copy link
Contributor

ernelio commented Jul 17, 2020

/merge qa

@ritchie-bot
Copy link
Contributor

ritchie-bot bot commented Jul 17, 2020

👌 Merged branch master into qa

@ernelio
Copy link
Contributor

ernelio commented Jul 17, 2020

/merge qa

@ritchie-bot
Copy link
Contributor

ritchie-bot bot commented Jul 17, 2020

👌 Merged branch master into qa

@ernelio
Copy link
Contributor

ernelio commented Jul 17, 2020

/merge qa

@ritchie-bot
Copy link
Contributor

ritchie-bot bot commented Jul 17, 2020

👌 Merged branch master into qa

@ernelio
Copy link
Contributor

ernelio commented Jul 17, 2020

/merge qa

@ritchie-bot
Copy link
Contributor

ritchie-bot bot commented Jul 17, 2020

👌 Merged branch master into qa

@ernelio
Copy link
Contributor

ernelio commented Jul 17, 2020

/merge qa

@ritchie-bot
Copy link
Contributor

ritchie-bot bot commented Jul 17, 2020

👌 Merged branch master into qa

@ernelio
Copy link
Contributor

ernelio commented Jul 17, 2020

/merge qa

@ritchie-bot
Copy link
Contributor

ritchie-bot bot commented Jul 17, 2020

👌 Merged branch master into qa

@ernelio
Copy link
Contributor

ernelio commented Jul 17, 2020

/merge qa

@ritchie-bot
Copy link
Contributor

ritchie-bot bot commented Jul 17, 2020

👌 Merged branch master into qa

@ernelio ernelio merged commit 04c4bbc into ZupIT:master Jul 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants