-
Notifications
You must be signed in to change notification settings - Fork 69
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
1406 [Transactions] Expected Value Dropdown Selection #1461
1406 [Transactions] Expected Value Dropdown Selection #1461
Conversation
} | ||
|
||
return ds, nil | ||
return [][]expression.DataStore{ds}, nil |
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.
Now we are returning an array of strings, this is to cover the scenario where there is no span id in the context and we can return the different resolved values from the selected spans.
setPrevRawExpression(props.expression || ''); | ||
}, | ||
[parseExpressionMutation, prevExpression, prevRawExpression] | ||
[context, parseExpressionMutation, prevContext, prevExpression, prevRawExpression] |
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.
Reruns if prev context changes
> | ||
<Editor type={SupportedEditors.Expression} placeholder="Expected Value" context={editorContext} /> | ||
</Form.Item> | ||
<AssertionCheckValue |
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.
Moved this to a diff file
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.
Looks good @xoscar great job!
@@ -11,7 +11,7 @@ import OperatorService from 'services/Operator.service'; | |||
import {TStructuredAssertion} from 'types/Assertion.types'; | |||
import {singularOrPlural} from 'utils/Common'; | |||
import AssertionCheckList from './AssertionCheckList'; | |||
import useAssertionFormValues from './hooks/useAssertionFormValues'; | |||
// import useAssertionFormValues from './hooks/useAssertionFormValues'; |
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.
👀
This PR adds a new feature for the test spec form by displaying a value list dropdown when selecting one of the attributes, it allows users to select with value to add to the expected section from the different selected spans.
Changes
Fixes
Checklist
https://www.loom.com/share/65e7cb843d9e4e108bb3901a1ef7b297