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

Fixes #37231 - Append domain name setting follow-up #10014

Merged

Conversation

Dyrkon
Copy link
Contributor

@Dyrkon Dyrkon commented Jan 24, 2024

Follow-up fixes.

@theforeman-bot
Copy link
Member

Can one of the admins verify this patch?

2 similar comments
@theforeman-bot
Copy link
Member

Can one of the admins verify this patch?

@theforeman-bot
Copy link
Member

Can one of the admins verify this patch?

app/models/host/base.rb Outdated Show resolved Hide resolved
app/models/host/base.rb Outdated Show resolved Hide resolved
@Dyrkon Dyrkon force-pushed the mm/append_domain_name_core_followup branch from e37997b to b58dcb2 Compare February 27, 2024 14:25
@Dyrkon Dyrkon force-pushed the mm/append_domain_name_core_followup branch 3 times, most recently from b47693b to 5c00136 Compare March 5, 2024 08:18
@stejskalleos
Copy link
Contributor

I see it's still in WIP, any blockers here?

@Dyrkon
Copy link
Contributor Author

Dyrkon commented Mar 5, 2024

I see it's still in WIP, any blockers here?

Yes, there is some internal dispute regarding the way we are going to resolve this.

@Dyrkon Dyrkon force-pushed the mm/append_domain_name_core_followup branch from 5c00136 to 4b800dd Compare March 7, 2024 08:29
@Dyrkon Dyrkon changed the title [WIP] Append domain name setting follow-up Fixes #37231 - Append domain name setting follow-up Mar 7, 2024
Copy link
Contributor

@stejskalleos stejskalleos left a comment

Choose a reason for hiding this comment

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

🍏 LGTM

Works as expected

@stejskalleos stejskalleos merged commit 1970292 into theforeman:develop Mar 27, 2024
35 of 36 checks passed
@stejskalleos
Copy link
Contributor

Thanks @Dyrkon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants