-
Notifications
You must be signed in to change notification settings - Fork 1k
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
[backend] Share/unshare tasks should be done by creation date from olders to more recents to have correctly ordered stream events (#8843)(#9200)(#9371) #9394
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #9394 +/- ##
==========================================
+ Coverage 65.22% 65.26% +0.03%
==========================================
Files 630 630
Lines 60199 60226 +27
Branches 6741 6752 +11
==========================================
+ Hits 39267 39306 +39
+ Misses 20932 20920 -12 ☔ View full report in Codecov by Sentry. |
4c550f6
to
2dce3d8
Compare
59ee949
to
2add871
Compare
ba9fab2
to
bb0bbc9
Compare
opencti-platform/opencti-graphql/tests/02-integration/04-manager/taskManager-test.ts
Outdated
Show resolved
Hide resolved
opencti-platform/opencti-graphql/tests/02-integration/04-manager/taskManager-test.ts
Outdated
Show resolved
Hide resolved
8b9a524
to
4ccaaca
Compare
...cti-platform/opencti-graphql/src/modules/attributes/internalObject-registrationAttributes.ts
Show resolved
Hide resolved
opencti-platform/opencti-graphql/tests/02-integration/04-manager/taskManager-test.ts
Outdated
Show resolved
Hide resolved
b414944
to
c7f4cb7
Compare
2f69a2b
to
8e2e5e3
Compare
1f4da88
to
f184a7d
Compare
} | ||
const message = `Update ${organizationNames} children in \`Shared with\``; | ||
const objectStoreBaseModified = { ...objectStoreBase, updated_at: utcDate() }; | ||
await storeUpdateEvent(context, user, objectStoreBase, objectStoreBaseModified, message, { allow_only_modified: true }); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Proposed changes
Other improvement:
Related issues
Checklist
Further comments