-
Notifications
You must be signed in to change notification settings - Fork 11
feat: Add data files support #510
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
`} | ||
> | ||
<Table.Cell>{file.name}</Table.Cell> | ||
<Table.Cell>Unique item per iteration</Table.Cell> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe Random item per iteration
is a bit more clear?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's not random 😬 it picks array values based on scenario.iterationInTest
as described here. I'll try to come up with better copy
<Tooltip | ||
content="Data file is referenced in a rule" | ||
hidden={!isFileInUse} | ||
> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤩
<Table.ColumnHeaderCell | ||
key={prop} | ||
uppercase={false} | ||
maxWidth="300px" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added maxWidth
to table columns to make sure it will look decent even with longer string values
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀 🚀 🚀
When importing a data file we don't jump into that view nor show a notification, I would possibly consider the notification of successful data file import 🤔
Closes #363 #507
Description
Add support for data files in parameterization (and custom code) rules. This is the first iteration of the feature.
Known issues
How to Test
Screenshots (if appropriate):