Skip to content

Allow to incorporate a path in entries #183

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

HHeda
Copy link

@HHeda HHeda commented Jul 19, 2023

Allows to enter entries in the form of {path/entry}, to access entries that are deeper in the tree or in an other branch

@MatthieuDartiailh
Copy link
Member

I would likely need to think more about this than I have the bandwidth to ATM. I understand that the access editor may not be the nicest thing to use but it allows to provide a nice autocompletion interface. Could you motivate this change ?

@HHeda
Copy link
Author

HHeda commented Jul 26, 2023

I modified this to access to the values of a loop outside of this loop. For example, I wanted to do something like this:

loop1

loop2

do something

save {loop1/loop2_loop_values}

Simply asking for {loop2_loop_values} does not work since the save task is not at the same node.

This worked as runtime but I needed to force the enqueue because the entry was not found. I didn't find another way but I might have missed something...

I then thought that starting the path from root could be enabled too, which led me to add a few options in the checks.

Unless I did a mistake, I only added new possible entries (even though they are not auto completed).

@MatthieuDartiailh
Copy link
Member

This can be achieved using the database access editor (select this tab and right click the entry you want to promote).

Please note that your solution does have value but I would need more time to consider what side consequences it may have.

@HHeda
Copy link
Author

HHeda commented Jul 27, 2023

Oh I didn't know this possibility, thank you !

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

Successfully merging this pull request may close these issues.

2 participants