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 several dnssec related zone options #158

Merged
merged 2 commits into from
May 12, 2020

Conversation

damluk
Copy link

@damluk damluk commented Mar 9, 2020

This is part of #156 and based on PR #157

@damluk damluk changed the title Feature/zone options/dnssec Introduce several dnssec related zone options Mar 9, 2020
@damluk damluk force-pushed the feature/zone-options/dnssec branch 2 times, most recently from 178fee9 to 41e63dd Compare March 12, 2020 08:14
@damluk damluk force-pushed the feature/zone-options/dnssec branch from 41e63dd to e21d6ae Compare April 8, 2020 16:04
Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

👍 once #157 is in.

@damluk damluk force-pushed the feature/zone-options/dnssec branch from e21d6ae to cbd4ae1 Compare April 9, 2020 22:01
@damluk damluk force-pushed the feature/zone-options/dnssec branch 2 times, most recently from e47edd7 to 13a3262 Compare May 7, 2020 19:10
@ekohl
Copy link
Member

ekohl commented May 11, 2020

This now needs a rebase since I've merged some of the other PRs and there's conflicts according to GH.

@damluk damluk force-pushed the feature/zone-options/dnssec branch from 13a3262 to a0de9c4 Compare May 11, 2020 16:26
@damluk damluk force-pushed the feature/zone-options/dnssec branch from a0de9c4 to af58fa5 Compare May 11, 2020 16:31
@ekohl ekohl merged commit 33d7c48 into theforeman:master May 12, 2020
@ekohl
Copy link
Member

ekohl commented May 12, 2020

Thanks!

@damluk damluk deleted the feature/zone-options/dnssec branch May 12, 2020 20:19
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