Skip to content
This repository was archived by the owner on Nov 3, 2022. It is now read-only.

Fixes #34095 - use report importer registry#98

Open
lzap wants to merge 1 commit intotheforeman:masterfrom
lzap:use-report-importer-reg-34095
Open

Fixes #34095 - use report importer registry#98
lzap wants to merge 1 commit intotheforeman:masterfrom
lzap:use-report-importer-reg-34095

Conversation

@lzap
Copy link
Member

@lzap lzap commented Dec 6, 2021

See theforeman/foreman#8958

Do not merge until the core PR is merged. Minimum Foreman version is now 3.2.

@ezr-ondrej
Copy link
Member

Core PR is in and I believe that this works even without the core PR so the minimum version is not necessary 🤷

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.

If the minimum Foreman version should be modified, line 50 (requires_foreman) should be modified too.

Signed-off-by: Lukas Zapletal <lzap+git@redhat.com>
@lzap
Copy link
Member Author

lzap commented Aug 25, 2022

Modified to 3.3 if that's what you mean.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants