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.

Sign up for GitHub

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

Jump to bottom

Cli overhaul: user password change #190

Merged
merged 2 commits into from
Sep 3, 2022
Merged

Cli overhaul: user password change #190

merged 2 commits into from
Sep 3, 2022

Conversation

Tinyblargon
Copy link
Collaborator

Another pull for #171

Work so far:

  • Made the config for the set user command optional, so this command can be used to change a users password.
  • Created integration tests for user password change function.
  • Made a change to the tests as Proxmox had a bug where it would ignore the value of the "expire" parameter. Which meant that the user would be disabled during the tests.

@mleone87 mleone87 merged commit 2d62d11 into Telmate:master Sep 3, 2022
frostyfab pushed a commit to frostyfab/proxmox-api-go that referenced this pull request Nov 17, 2023
@mleone87
Merge pull request Telmate#190 from Tinyblargon/CLI-Overhaul
02e958f 
Cli overhaul: user password change
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Tinyblargon @mleone87