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

Fixed/updated URL for “Passing Variables on the Command Line" link. #14763

Merged
merged 1 commit into from Jan 19, 2024

Conversation

tvo318
Copy link
Member

@tvo318 tvo318 commented Jan 12, 2024

SUMMARY

Porting changes from Controller documentation to fix broken link in the Job Templates chapter of the User Guide.

ISSUE TYPE
  • Bug, Docs Fix or other nominal change
COMPONENT NAME
  • Docs
AWX VERSION

latest

@oraNod
Copy link
Collaborator

oraNod commented Jan 15, 2024

@tvo318 Hey this looks great and it's super cool to see that an issue that originated downstream gets ported to community docs! 🎉

Just one thing to note, it might be preferable to avoid linking Jira tickets in GH issues. Community users can't access those details. IMO it's also a best practice to avoid potentially leaking sensitive details like customer info or vulnerabilities. On other projects where a downstream issue goes to community I've put in a generic description like "Porting changes from Controller documentation". It's still good to make it obvious the change originates downstream. Over time all those little changes add up and it helps to show how community wins!

@tvo318 tvo318 merged commit 74cd23b into ansible:devel Jan 19, 2024
21 checks passed
@tvo318 tvo318 deleted the linkfix branch January 29, 2024 21:40
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