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

add --no-email for disabling email in ACME query #2060

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

add --no-email for disabling email in ACME query #2060

wants to merge 5 commits into from

Conversation

pschou
Copy link
Contributor

@pschou pschou commented Dec 5, 2023 •
edited by ldez

Fixes #277
Closes #1378

@pschou
add --no-email for disabling email in ACME query
6a0a788 
Signed-off-by: schou <pschou@users.noreply.github.com>
@ldez ldez closed this Dec 5, 2023
@ldez ldez reopened this Dec 5, 2023
@ldez ldez self-requested a review December 5, 2023 19:11
@ldez ldez added the area/cli label Dec 5, 2023
@pschou
update error message for helpful hint
0bae2c9 
Signed-off-by: schou <pschou@users.noreply.github.com>
@pschou
wording
e5f135c 
Signed-off-by: schou <pschou@users.noreply.github.com>
@pschou
usage helpful hint
504497e 
Signed-off-by: schou <pschou@users.noreply.github.com>
@pschou
return empty string if no-email is set
51ef22a 
Signed-off-by: schou <pschou@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ldez ldez Awaiting requested review from ldez

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
area/cli enhancement
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

register without email
2 participants
@pschou @ldez