Skip to content
This repository was archived by the owner on Dec 5, 2025. It is now read-only.

Conversation

@JeremyCloarec
Copy link
Contributor

@JeremyCloarec JeremyCloarec commented Dec 13, 2024

Proposed changes

  • add draft context to current work when sending bundle to ingestion in a draft

Related issues

Checklist

  • I consider the submitted work as finished
  • I tested the code for its functionality
  • I wrote test cases for the relevant uses case
  • I added/update the relevant documentation (either on github or on notion)
  • Where necessary I refactored code to improve the overall quality

Further comments

If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

@JeremyCloarec JeremyCloarec added the multi-repository When a PR requires several repository changes label Dec 13, 2024
@JeremyCloarec JeremyCloarec added the filigran team use to identify PR from the Filigran team label Dec 13, 2024
@JeremyCloarec JeremyCloarec changed the title Opencti/issue/6577chunk UI [client] add draft context to work when sending bundle in draft (opencti #6577) Dec 18, 2024
@JeremyCloarec JeremyCloarec marked this pull request as ready for review December 18, 2024 13:23
@JeremyCloarec JeremyCloarec force-pushed the opencti/issue/6577chunkUI branch from 81996f5 to 9f8d6d3 Compare December 18, 2024 14:34
if bundle_send_to_queue:
if work_id:
self.api.work.add_expectations(work_id, expectations_number)
if draft_id:
Copy link
Member

Choose a reason for hiding this comment

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

move the condition up under if work_id

@JeremyCloarec JeremyCloarec merged commit a78975b into release/6.5.0 Dec 19, 2024
4 checks passed
@JeremyCloarec JeremyCloarec deleted the opencti/issue/6577chunkUI branch December 19, 2024 16:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

filigran team use to identify PR from the Filigran team multi-repository When a PR requires several repository changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants