Skip to content

Add to docs: YAML configuration, dynamic UI, triggers, engine placement #1749

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Add to docs: YAML configuration, dynamic UI, triggers, engine placement #1749

wants to merge 7 commits into from

Conversation

RosenbergYehuda
Copy link
Contributor

@RosenbergYehuda RosenbergYehuda commented May 12, 2025 •
edited
Loading

Status

Ready/In Progress/In Hold (Reason for hold)

Related Issues

fixes: link to the issue

Description

Added documentation for new features for the integration yml.

Screenshots

Paste here any images that will help the reviewer

@RosenbergYehuda RosenbergYehuda self-assigned this May 12, 2025
@content-bot
Copy link
Collaborator

Preview Site Available

Congratulations! The automatic build has completed successfully.
A preview site is available at: https://xsoar-pan-dev--pull-request-1749-esv6mauu.web.app

Important: Make sure to inspect your changes at the preview site.

3 similar comments
@content-bot
Copy link
Collaborator

Preview Site Available

Congratulations! The automatic build has completed successfully.
A preview site is available at: https://xsoar-pan-dev--pull-request-1749-esv6mauu.web.app

Important: Make sure to inspect your changes at the preview site.

@content-bot
Copy link
Collaborator

Preview Site Available

Congratulations! The automatic build has completed successfully.
A preview site is available at: https://xsoar-pan-dev--pull-request-1749-esv6mauu.web.app

Important: Make sure to inspect your changes at the preview site.

@content-bot
Copy link
Collaborator

Preview Site Available

Congratulations! The automatic build has completed successfully.
A preview site is available at: https://xsoar-pan-dev--pull-request-1749-esv6mauu.web.app

Important: Make sure to inspect your changes at the preview site.

@content-bot
Copy link
Collaborator

Preview Site Available

Congratulations! The automatic build has completed successfully.
A preview site is available at: https://xsoar-pan-dev--pull-request-1749-esv6mauu.web.app

Important: Make sure to inspect your changes at the preview site.

tschanfeld
tschanfeld reviewed May 14, 2025
View reviewed changes
tschanfeld
tschanfeld reviewed May 14, 2025
View reviewed changes
tschanfeld
tschanfeld reviewed May 14, 2025
View reviewed changes
docs/integrations/dynamic-ui-fields.md Outdated
section: Connect
```

**Important Note:** This functionality requires a compatible UI version. Using `engine_placeholder` in an outdated UI will create a non-functional parameter with the name `engine_placeholder`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we want to say from which UI version?

tschanfeld
tschanfeld reviewed May 14, 2025
View reviewed changes
tschanfeld
tschanfeld reviewed May 14, 2025
View reviewed changes
@content-bot
Copy link
Collaborator

Preview Site Available

Congratulations! The automatic build has completed successfully.
A preview site is available at: https://xsoar-pan-dev--pull-request-1749-esv6mauu.web.app

Important: Make sure to inspect your changes at the preview site.

@stale Stale
Copy link

stale bot commented May 30, 2025

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs.

If you'd like to keep it open, please leave a comment with the status of the PR.

Thank you for your contribution!

@stale stale bot added the stale label May 30, 2025
tschanfeld
tschanfeld reviewed Jun 10, 2025
View reviewed changes
Copy link
Contributor

@tschanfeld tschanfeld left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we still want to merge this?

@RosenbergYehuda
Copy link
Contributor Author

@tschanfeld - Yes

@tschanfeld
Update dynamic-ui-fields.md
6cb52a5 
edited the note re: UI version
tschanfeld
tschanfeld reviewed Jun 10, 2025
View reviewed changes
Copy link
Contributor

@tschanfeld tschanfeld left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor edit

@content-bot
Copy link
Collaborator

Preview Site Available

Congratulations! The automatic build has completed successfully.
A preview site is available at: https://xsoar-pan-dev--pull-request-1749-98z5vbwh.web.app

Important: Make sure to inspect your changes at the preview site.

RosenbergYehuda and others added 2 commits June 12, 2025 18:01
@RosenbergYehuda
fix names
c4f4e5e
@julieschwartz18 @RosenbergYehuda
Update dev-setup.md (#1746)
7672805 
Under Bootstrap, added a step for running the poetry self add poetry-plugin-shell script
@content-bot
Copy link
Collaborator

Preview Site Available

Congratulations! The automatic build has completed successfully.
A preview site is available at: https://xsoar-pan-dev--pull-request-1749-98z5vbwh.web.app

Important: Make sure to inspect your changes at the preview site.

Detected modified urls:

@stale Stale
Copy link

stale bot commented Jun 26, 2025

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs.

If you'd like to keep it open, please leave a comment with the status of the PR.

Thank you for your contribution!

@stale stale bot added the stale label Jun 26, 2025
@stale stale bot removed the stale label Jul 3, 2025
@content-bot
Copy link
Collaborator

Preview Site Available

Congratulations! The automatic build has completed successfully.
A preview site is available at: https://xsoar-pan-dev--pull-request-1749-k14lfey9.web.app

Important: Make sure to inspect your changes at the preview site.

Detected modified urls:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tschanfeld tschanfeld tschanfeld approved these changes

@JasBeilin JasBeilin Awaiting requested review from JasBeilin

Assignees

@RosenbergYehuda RosenbergYehuda

Labels
docs-approved
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@RosenbergYehuda @content-bot @tschanfeld @julieschwartz18