Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

Not updating registration_sent value when updating user.#5452

Merged
mitchell852 merged 5 commits intoapache:masterfrom
rimashah25:bugfix/CDN-11825
Jan 25, 2021
Merged

Not updating registration_sent value when updating user.#5452
mitchell852 merged 5 commits intoapache:masterfrom
rimashah25:bugfix/CDN-11825

Conversation

@rimashah25
Copy link
Contributor

@rimashah25 rimashah25 commented Jan 21, 2021

What does this PR (Pull Request) do?

Which Traffic Control components are affected by this PR?

  • Traffic Ops

What is the best way to verify this PR?

Run API tests

If this is a bug fix, what versions of Traffic Control are affected?

master (9930731)

The following criteria are ALL met by this PR

  • I have explained why tests are unnecessary
  • I have explained why documentation is unnecessary
  • This PR includes an update to CHANGELOG.md OR such an update is not necessary
  • This PR includes any and all required license headers
  • This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Additional Information

No change in logic to update tests and documentation

@rimashah25 rimashah25 marked this pull request as ready for review January 21, 2021 22:05
@mitchell852 mitchell852 self-assigned this Jan 22, 2021
@rimashah25 rimashah25 changed the title Setting registration_sent to current time when updating user. Not updating registration_sent value when updating user. Jan 22, 2021
Copy link
Member

@mitchell852 mitchell852 left a comment

Choose a reason for hiding this comment

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

lgtm

@mitchell852 mitchell852 merged commit 07554d8 into apache:master Jan 25, 2021
@rimashah25 rimashah25 deleted the bugfix/CDN-11825 branch January 25, 2021 20:22
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.

PUT /users/:id sets tm_user.registration_sent (date) to null

2 participants