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 #25787 - Make login and taxonomy settings optional #707

Merged
merged 1 commit into from
Jan 13, 2019

Conversation

tbrisker
Copy link
Member

@tbrisker tbrisker commented Jan 6, 2019

No description provided.

@tbrisker
Copy link
Member Author

tbrisker commented Jan 6, 2019

@ekohl - i hope this is what you meant in #690 (comment) ?

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.

Please add a note in the compatibility table in README.md.

manifests/init.pp Outdated Show resolved Hide resolved
manifests/params.pp Outdated Show resolved Hide resolved
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.

Going to let the tests finish.

# $organizations_enabled:: Enable organizations? (Unused since 1.21)
#
# $authentication:: Enable user authentication. (Unused since 1.21)
# Initial credentials are set using admin_username and admin_password.
Copy link
Member

Choose a reason for hiding this comment

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

Looks like there was also an oversight here: these are now prefixed with initial_.

Copy link
Member Author

Choose a reason for hiding this comment

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

fixed on the rebase

@ekohl
Copy link
Member

ekohl commented Jan 11, 2019

There's a merge conflict now :(

@tbrisker
Copy link
Member Author

@ekohl thanks, rebased on latest master now.

manifests/init.pp Outdated Show resolved Hide resolved
@mmoll
Copy link
Contributor

mmoll commented Jan 13, 2019

@tbrisker one tiny thing :)

@mmoll mmoll merged commit f404fb1 into theforeman:master Jan 13, 2019
@mmoll
Copy link
Contributor

mmoll commented Jan 13, 2019

merged, thanks @tbrisker!

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.

None yet

4 participants