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

Update `with_subelements` to handle sub-lists located deeper in dict #9005

Closed
wants to merge 1 commit into from
Closed

Update `with_subelements` to handle sub-lists located deeper in dict #9005

wants to merge 1 commit into from

Conversation

@jumoel
Copy link

@jumoel jumoel commented Sep 15, 2014

Before, with_subelements could only handle that the nested list was one element 'deep' compared to the first argument.

Now, with_subelements accepts the second parameter in the form of foo.bar.baz, which will get the list at foo.bar.baz instead of complaining that the literal key foo.bar.baz doesn't exist.

Before, `with_subelements` could only handle that the nested list was one element 'deep' compared to the first argument.

Now, `with_subelements` accepts the second parameter in the form of `foo.bar.baz`, which will get the list at `foo.bar.baz` instead of complaining that the literal key `foo.bar.baz` doesn't exist.
@jumoel jumoel changed the title Update `with_subelements` to deeper located sub-lists Update `with_subelements` to handle sub-lists located deeper in dict Sep 15, 2014
@jimi-c jimi-c added P3 labels Sep 15, 2014
@dmkoch
Copy link
Contributor

@dmkoch dmkoch commented Jan 14, 2015

+1 This would be extremely useful.

@podung
Copy link

@podung podung commented Jan 29, 2015

+1 We are utilizing the change from the PR. Would love to see this pull request accepted.

@cadetstar
Copy link

@cadetstar cadetstar commented Jan 29, 2015

+1 Would be handy.

@johnalla0
Copy link

@johnalla0 johnalla0 commented Jan 29, 2015

+1, we need this

@alanflorendo
Copy link

@alanflorendo alanflorendo commented Jan 29, 2015

+1 yes please

@dylanjw
Copy link

@dylanjw dylanjw commented Jun 4, 2015

+1

@jumoel
Copy link
Author

@jumoel jumoel commented Jun 4, 2015

Something similar seems to have been merged into what will be Ansible v2: #10995.

@jumoel jumoel closed this Jun 4, 2015
@ansibot ansibot added feature and removed feature_pull_request labels Mar 4, 2018
@ansible ansible locked and limited conversation to collaborators Apr 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

9 participants