-
Notifications
You must be signed in to change notification settings - Fork 585
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
Object repository datatable as variables, and datatable count with iteration - done #953
Comments
Hi Marcelo! You can specify the datatable using Above error and file naming is probably you are referring to the old documentation. For current doc - https://tagui.readthedocs.io/en/latest/advanced.html#datatables Also, to use commas |
Oh I get the problem now, the `varstring` works because it will retrieve data from your csv file. But using varstring directly in the condition, TagUI assumes that it is a variable and doesn't read from the csv file the value. Let me chew on this what is the best solution, and if we can make a patch to solve this. CC @ruthtxh (think the issue you ran into previously with csv_row()) |
Yes, I don't think there's a direct solution now, the parsing for the backticks happens before script runs and only for normal steps, it does not parse for lines that are in JS code, because variables would not have `` in that context. Will think over. |
A workaround in the meantime is below, after the 2 steps below, varstring can be used in the if condition -
|
If this works, you can use this workaround for the time being. In the next release, I'll see if there's a good permanent fix. |
it works ! |
Welcome Marcelo, thanks for raising this to us! Your feedback helps us to improve TagUI. Reopening issue to investigate on a permanent solution for next release. Cc @ruthtxh |
Above commit works with below examples. If a field has spaces, the spaces will be replaced with underscore for valid variable names. If there is single quote in the field data, it will be escaped automatically to not break JavaScript compilation. You can download the latest copy of TagUI with this and unzip to overwrite your existing installation (please drag the folders under tagui\src to overwrite your existing installation) - https://github.com/kelaberetiv/TagUI/archive/master.zip In the next release, this will become part of the packaged zip files. repository.tag
tagui_local.csv
datatable.tag
data.csv
Fyi @ruthtxh |
To review if defining all datatable and object repository values is better, instead of defining what is being used in TagUI steps. |
Above commit is v2 of this enhancement, it will process all fields regardless of whether a field is used in TagUI steps. repository.tag
tagui_local.csv
datatable.tag
data.csv
|
Users can download the latest copy of TagUI with this and unzip to overwrite your existing installation (please drag the folders under tagui\src to overwrite your existing installation) - https://github.com/kelaberetiv/TagUI/archive/master.zip In the next release, this will become part of the packaged zip files. |
Also, as part of the change, following will be the new way to access iteration (which iteration run in a datatable) - new way
old way
|
Above commit adds v3 of this feature, to make it robust for edges. Otherwise it is possible for some header names to break the execution due to invalid JavaScript variable naming convention. Users can download the latest copy of TagUI with this and unzip to overwrite your existing installation (please drag the folders under tagui\src to overwrite your existing installation) - https://github.com/kelaberetiv/TagUI/archive/master.zip In the next release, this will become part of the packaged zip files. |
tested ok for - datatable.csv
tagui_local.csv
|
Closing issue, change included in latest packaged release - https://github.com/kelaberetiv/TagUI/releases/tag/v6.46.0 |
Hello
attachment error is occurring
The text was updated successfully, but these errors were encountered: