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

Reconsider harvest mapping recusrion #685

Open
atruskie opened this issue Oct 8, 2024 · 0 comments
Open

Reconsider harvest mapping recusrion #685

atruskie opened this issue Oct 8, 2024 · 0 comments

Comments

@atruskie
Copy link
Member

atruskie commented Oct 8, 2024

A repeated frustration users have reported is that it does not make sense for the values of the mappings to not recurse.

We chose the strategy of choosing the first mapping record that matches a path and then we use the values for the folder.

But we should instead consider doing this search for each mapping for each value - continuing to recurse if we find a non-null value.

Here is an example where a user expected the values to recurse and contacted us to work out what was wrong:

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant