-
Notifications
You must be signed in to change notification settings - Fork 7
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
Support the notifier job configuration #618
Comments
Can I ask you about the
|
ORT run creation form is already over 1000 lines long, so we're probably going to do some refactoring to it before adding new configurations. |
Good point, they are treated as secrets, so the values are paths to secrets in the secret storage. |
Thanks, so it's safe for us to expose them in the form as ordinary text inputs. |
Add support for the Notifier by adding an accordion component for it to the run creation form. The Notifier is disabled by default. Resolves eclipse-apoapsis#618. Signed-off-by: Jyrki Keisala <[email protected]>
Add support for the Notifier by adding an accordion component for it to the run creation form. The Notifier is disabled by default. Resolves eclipse-apoapsis#618. Signed-off-by: Jyrki Keisala <[email protected]>
Add support for the Notifier by adding an accordion component for it to the run creation form. The Notifier is disabled by default. Resolves eclipse-apoapsis#618. Signed-off-by: Jyrki Keisala <[email protected]>
Add support for the Notifier by adding an accordion component for it to the run creation form. The Notifier is disabled by default. Resolves #618. Signed-off-by: Jyrki Keisala <[email protected]>
Add support for the notifier job configuration to the form to create a new ORT run. Also show the notifier job configuration in the run details.
The text was updated successfully, but these errors were encountered: