Skip to content

Fix Foreman legacy UI erroring out#92

Merged
timogoebel merged 3 commits intotheforeman:masterfrom
ananace:fix-legacy-ui
May 31, 2024
Merged

Fix Foreman legacy UI erroring out#92
timogoebel merged 3 commits intotheforeman:masterfrom
ananace:fix-legacy-ui

Conversation

@ananace
Copy link
Member

@ananace ananace commented May 2, 2023

Fixes #90

@ekohl
Copy link
Member

ekohl commented Dec 28, 2023

In #94 I'm adding it to the modern UI instead, but it's still lacking actions.

@laugmanuel
Copy link
Member

I would support fixing this as long as modern UI is not implemented. This seems to be a quick fix for now.

@ekohl
Copy link
Member

ekohl commented May 17, 2024

There's a merge conflict, so please rebase.

@ananace ananace force-pushed the fix-legacy-ui branch 3 times, most recently from 3653261 to 56833bb Compare May 17, 2024 13:49
@laugmanuel
Copy link
Member

@dgoetz do you mind having a look? I would also purpose a new release after that because the current plugin version is broken on multiple parts. I could take care of packaging PRs

timogoebel and others added 2 commits May 31, 2024 10:43
Co-authored-by: Ewoud Kohl van Wijngaarden <ewoud@kohlvanwijngaarden.nl>
This reverts commit 26da44b.
@timogoebel timogoebel merged commit ff8cf19 into theforeman:master May 31, 2024
@timogoebel
Copy link
Member

Thanks, @ananace.

@ananace ananace deleted the fix-legacy-ui branch May 31, 2024 08:57
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.

undefined method `host_monitoring_result_icon_class' for #<ActionView::Base:0x000000001698d8> Did you mean? host_config_reports_url

5 participants