-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Implement regex validators for text workflow parameter #18781
Conversation
xref: galaxyproject/training-material#5288 (comment) (maybe we can add |
74c12eb
to
2c31cc1
Compare
regular expression sounds good to me, but we're using the tool form builder here (for better or worse), do you see a way to embed terms here ? to the form builder this just looks like a regular text parameter. |
Maybe it's time to have a regex parameter, where we can provide more help ? That concept could be extended to awk, ecmascript, python etc |
Which allows building repeats and conditionals.
We don't need to replciate this logic just for workflow parameters.
2bc087b
to
1bbfc9c
Compare
Finished in #19092 |
Can't quite get the repeat to render in the editor when loading the persisted state ... but other than that I think this is functional. Will also need galaxyproject/gxformat2#105 for tests
How to test the changes?
(Select all options that apply)
License