Skip to content

Remove session phone attempt count initialization#5343

Merged
aduth merged 1 commit intomainfrom
aduth-remove-phone-throttle-session
Sep 3, 2021
Merged

Remove session phone attempt count initialization#5343
aduth merged 1 commit intomainfrom
aduth-remove-phone-throttle-session

Conversation

@aduth
Copy link
Copy Markdown
Contributor

@aduth aduth commented Aug 27, 2021

Builds on #5339 to remove session values unused after changes proposed in that branch, left temporarily to avoid deploy issues with live sessions.

Should only be merged after deploy including #5339.

Additional context: #5339 (comment)

Copy link
Copy Markdown
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

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

LGTM

Base automatically changed from aduth-phone-throttle-refactor to main August 27, 2021 17:34
@aduth aduth force-pushed the aduth-remove-phone-throttle-session branch from d9c0a71 to 355f22c Compare August 27, 2021 19:41
@aduth
Copy link
Copy Markdown
Contributor Author

aduth commented Sep 3, 2021

Merging, since #5339 went live to production yesterday.

@aduth aduth merged commit 1c17d5d into main Sep 3, 2021
@aduth aduth deleted the aduth-remove-phone-throttle-session branch September 3, 2021 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants