Skip to content

Conversation

@raviks789
Copy link
Collaborator

Other available fields should be suggested when no check command has been chosen for service template.

fixes #2826, #2815

@cla-bot cla-bot bot added the cla/signed label Dec 5, 2023
@RincewindsHat
Copy link
Member

RincewindsHat commented Dec 5, 2023

does not work for notifications on a first test

@raviks789 raviks789 force-pushed the fix/form-field-suggestion branch from 3e561a2 to 6cbb9ec Compare December 5, 2023 13:09
@raviks789
Copy link
Collaborator Author

raviks789 commented Dec 6, 2023

does not work for notifications on a first test

As discussed offline, this is a fix for the bug where the data fields are not available for selection when a check command is not selected (Not a feature to show the custom properties of notification command in the notifications form).

@raviks789 raviks789 marked this pull request as ready for review December 6, 2023 09:32
@raviks789 raviks789 requested a review from nilmerg December 6, 2023 09:32
@raviks789 raviks789 self-assigned this Dec 6, 2023
@raviks789 raviks789 force-pushed the fix/form-field-suggestion branch 2 times, most recently from 3c9169f to 7001316 Compare December 18, 2023 08:52
@bigfreak85
Copy link

works and has helped me to get the job done

@raviks789 raviks789 force-pushed the fix/form-field-suggestion branch 3 times, most recently from f8ebda1 to 6f593d3 Compare January 10, 2024 16:49
@raviks789 raviks789 force-pushed the fix/form-field-suggestion branch 9 times, most recently from 7d96a36 to ff470b1 Compare January 15, 2024 12:48
@nilmerg nilmerg added this to the v1.11.1 milestone Jan 16, 2024
@nilmerg nilmerg added the bug label Jan 16, 2024
@raviks789 raviks789 requested a review from nilmerg January 16, 2024 11:37
Other available fields should be suggested when no check command has been
chosen for service template.
@raviks789 raviks789 force-pushed the fix/form-field-suggestion branch from ff470b1 to 1ec1698 Compare January 16, 2024 11:38
@raviks789 raviks789 force-pushed the fix/form-field-suggestion branch from 1ec1698 to 8a1588f Compare January 16, 2024 13:28
@raviks789 raviks789 force-pushed the fix/form-field-suggestion branch from 8a1588f to 1788d80 Compare January 16, 2024 13:38
@raviks789 raviks789 force-pushed the fix/form-field-suggestion branch from 1788d80 to 80d566f Compare January 16, 2024 14:30
@raviks789 raviks789 requested a review from nilmerg January 16, 2024 14:37
@nilmerg nilmerg merged commit 0493f93 into master Jan 16, 2024
@nilmerg nilmerg deleted the fix/form-field-suggestion branch January 16, 2024 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Problem New custom field in Notification Templates

5 participants