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

letsencrypt: Add desec.io support DNS challenge #3157

Merged

Conversation

henrikhierl
Copy link
Contributor

No description provided.

@home-assistant
Copy link

home-assistant bot commented Aug 1, 2023

Hi @henrikhierl

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

Copy link
Contributor

@mdegat01 mdegat01 left a comment

Choose a reason for hiding this comment

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

Missing a changelog and version bump. Looks good otherwise.

@home-assistant
Copy link

home-assistant bot commented Sep 1, 2023

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

@home-assistant home-assistant bot marked this pull request as draft September 1, 2023 15:14
@kmahelona
Copy link

+1 to getting this PR merged. 🙏🏽

@agners
Copy link
Member

agners commented Nov 24, 2023

@henrikhierl the add-on now got updated to the latest Alpine version etc. Can resolve the merge conflict and add the version bump to the changelog and config.yaml (similar to how #3152 was done)? Then we should be able to merge this soon.

@henrikhierl henrikhierl force-pushed the add_letsencrypt_desec_support branch from 1cb93b5 to 24159fa Compare November 28, 2023 13:06
@henrikhierl
Copy link
Contributor Author

@agners just rebased the branch and add the version bump to the changelog and config.yaml

@agners agners marked this pull request as ready for review November 28, 2023 15:45
@home-assistant home-assistant bot requested a review from mdegat01 November 28, 2023 15:46
@agners agners merged commit 622e980 into home-assistant:master Dec 4, 2023
9 checks passed
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.

4 participants