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.

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

Introduce separate UpdateRegistrationContact & UpdateRegistrationKey methods in RA & SA #7735

Merged
merged 12 commits into from
Nov 6, 2024

Conversation

jprenken
Copy link
Contributor

@jprenken jprenken commented Oct 2, 2024

Introduce separate UpdateRegistrationContact & UpdateRegistrationKey methods in RA & SA

Clear contact field during DeactivateRegistration

Part of #7716
Part of #5554

…methods in RA & SA

Clear contact field in DeactivateRegistration

Part of #7716
Part of #5554
ra/proto/ra.proto Outdated Show resolved Hide resolved
ra/ra.go Outdated Show resolved Hide resolved
sa/proto/sa.proto Outdated Show resolved Hide resolved
sa/sa.go Show resolved Hide resolved
sa/sa.go Show resolved Hide resolved
@jprenken jprenken marked this pull request as ready for review October 26, 2024 23:46
@jprenken jprenken requested a review from a team as a code owner October 26, 2024 23:46
@jprenken jprenken requested review from jsha and aarongable October 26, 2024 23:46
ra/ra.go Show resolved Hide resolved
ra/ra_test.go Outdated Show resolved Hide resolved
sa/sa.go Outdated Show resolved Hide resolved
sa/sa.go Outdated Show resolved Hide resolved
sa/sa_test.go Show resolved Hide resolved
sa/sa_test.go Show resolved Hide resolved
@jprenken jprenken marked this pull request as draft November 5, 2024 08:45
@beautifulentropy beautifulentropy marked this pull request as ready for review November 5, 2024 15:18
@jprenken jprenken merged commit 6a2819a into main Nov 6, 2024
12 checks passed
@jprenken jprenken deleted the updatereg-1 branch November 6, 2024 18:07
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.

3 participants