Skip to content
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

ObjectSuggestions: Make CustomVar suggestion sources adjustable #1100

Merged
merged 3 commits into from
Jan 20, 2025

Conversation

sukhwinder33445
Copy link
Contributor

@sukhwinder33445 sukhwinder33445 commented Nov 26, 2024

The ObjectSuggestions instance with DependencyNode as the model can only retrieve custom variables for Host, Service, Hostgroup and Servicegroup. Other relations are not defined and lead to an error.

@sukhwinder33445 sukhwinder33445 self-assigned this Nov 26, 2024
@cla-bot cla-bot bot added the cla/signed CLA is signed by all contributors of a PR label Nov 26, 2024
@sukhwinder33445 sukhwinder33445 force-pushed the fix/object-customvar-suggestions branch 2 times, most recently from 98fc7c8 to 5adb282 Compare November 26, 2024 14:52
@sukhwinder33445 sukhwinder33445 force-pushed the fix/object-customvar-suggestions branch from 5adb282 to b7e8c20 Compare January 10, 2025 10:35
@sukhwinder33445 sukhwinder33445 force-pushed the fix/object-customvar-suggestions branch from b7e8c20 to c445235 Compare January 16, 2025 14:37
@sukhwinder33445 sukhwinder33445 force-pushed the fix/object-customvar-suggestions branch from c445235 to f7e2c4c Compare January 16, 2025 14:42
@nilmerg nilmerg force-pushed the fix/object-customvar-suggestions branch from f7e2c4c to 2e0ed59 Compare January 20, 2025 09:50
@nilmerg nilmerg merged commit 9eea87d into dependencies Jan 20, 2025
22 checks passed
@nilmerg nilmerg deleted the fix/object-customvar-suggestions branch January 20, 2025 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla/signed CLA is signed by all contributors of a PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants