-
Notifications
You must be signed in to change notification settings - Fork 27
Sidepanel supports <input> fields for user input through plugins #195
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.
Nice work
Co-authored-by: Hendrik Eeckhaut <[email protected]>
Co-authored-by: Hendrik Eeckhaut <[email protected]>
@heeckhau can you give this another review with the updated changes |
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.
Approved, except for one remark
So this only supports input parameters on the first step? I was slightly confused by this because you can configure inputs on any step in config.json tlsn-extension/src/entries/SidePanel/SidePanel.tsx Lines 279 to 283 in 8664726
|
No description provided.