Skip to content

brk: Removed change password endpoints #72

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

Merged
merged 1 commit into from
Mar 12, 2018

Conversation

Dorthu
Copy link
Collaborator

@Dorthu Dorthu commented Feb 22, 2018

No description provided.

Copy link
Collaborator

@yoshizzle yoshizzle left a comment

Choose a reason for hiding this comment

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

I will approve these on their merit for completing the task, but I will also reiterate the comment I made on the API side of this task:

I don't see why we are removing this ability and still keeping in the ability to set a password via the API for new users on an account. Seems to me that if we are removing it in one place it should be removed in all places.

That said, the work done is technically correct, hence my approval.

@jamesottinger jamesottinger merged commit ea774de into linode:develop Mar 12, 2018
alioso pushed a commit that referenced this pull request Aug 22, 2019
Hopefully the last of the CLI additions
LBGarber pushed a commit to LBGarber/linode-api-docs that referenced this pull request Jun 15, 2022
Cloud Manager 0.36.0 changelog, update readme instructions
LBGarber pushed a commit to LBGarber/linode-api-docs that referenced this pull request Jun 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants