-
Notifications
You must be signed in to change notification settings - Fork 98
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
Add HTTP(S) proxy username and password Setting #3635
Conversation
The PR preview for b9fe65d is available at theforeman-foreman-documentation-preview-pr-3635.surge.sh The following output files are affected by this PR: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small suggestion on the wording of descriptions if you agree
d7f5d12
to
b9fe65d
Compare
Why was this merged before the change itself was merged? This needs to be reverted, especially in 3.14. |
|
We have a label for pending code: |
What changes are you introducing?
Why are you introducing these changes? (Explanation, links to references, issues, etc.)
theforeman/foreman#10432
Anything else to add? (Considerations, potential downsides, alternative solutions you have explored, etc.)
Checklists
Please cherry-pick my commits into: