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

Slack bb fixes #31393

Merged
merged 20 commits into from
Dec 17, 2023
Merged

Slack bb fixes #31393

merged 20 commits into from
Dec 17, 2023

Conversation

amshamah419
Copy link
Contributor

@amshamah419 amshamah419 commented Dec 10, 2023

Status

  • In Progress
  • Ready
  • In Hold - (Reason for hold)

Related Issues

fixes: https://jira-dc.paloaltonetworks.com/browse/XSUP-31182
fixes: https://jira-dc.paloaltonetworks.com/browse/XSUP-31179
fixes: https://jira-dc.paloaltonetworks.com/browse/XSUP-28661

Description

SBB was using the internal API to post updates to an incident's context. We've changed the implementation to remove the need to the internal API call.

Must have

  • Tests
  • Documentation

@amshamah419 amshamah419 self-assigned this Dec 10, 2023
Copy link
Contributor

@yuvalbenshalom yuvalbenshalom left a comment

Choose a reason for hiding this comment

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

looks good, some comments:

  1. please add documentation
  2. please add unit test to cover this flow(noticed no test broken in the change)
  3. see my comments.

@@ -0,0 +1,4 @@
{
"breakingChanges": true,
"breakingChangesNotes": "SlackBlockBuilder no longer relies on the XSOAR API in order to update an incident's context. As such, playbooks which currently use the SlackBlockBuilder script should be modified as described in the documentation."
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we should name the location of the relevant documentation. like "readme, sub section {name}"

Copy link
Contributor

@yuvalbenshalom yuvalbenshalom left a comment

Choose a reason for hiding this comment

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

looks good. see may comments.

@@ -0,0 +1,4 @@
{
"breakingChanges": true,
"breakingChangesNotes": "- SlackBlockBuilder no longer relies on the XSOAR API in order to update an incident's context. As such, playbooks which currently use the SlackBlockBuilder script should be modified as described in the documentation.\n- Additional information regarding the change and migrating to pack version 3.3.0 can be found [here.](https://xsoar.pan.dev/docs/reference/scripts/slack-block-builder#troubleshooting)"
Copy link
Contributor

Choose a reason for hiding this comment

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

need to test that this link is working on the popup before merge.

@amshamah419 amshamah419 merged commit 3b8b5b5 into master Dec 17, 2023
@amshamah419 amshamah419 deleted the slack-bb-fixes branch December 17, 2023 13:23
sapirshuker pushed a commit that referenced this pull request Dec 21, 2023
maimorag pushed a commit that referenced this pull request Dec 31, 2023
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.

2 participants