-
Notifications
You must be signed in to change notification settings - Fork 946
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
[Frontend/Backend] split cpapabilites Data Sharing & Ingestion into 3 + manage CSV + Bypass ref + Connector Api (#7428) #7149
Conversation
ef23535
to
44078c3
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #7149 +/- ##
==========================================
+ Coverage 67.45% 67.49% +0.03%
==========================================
Files 561 561
Lines 69236 69255 +19
Branches 5867 5908 +41
==========================================
+ Hits 46705 46741 +36
+ Misses 22531 22514 -17 ☔ View full report in Codecov by Sentry. |
...m/opencti-graphql/src/migrations/1716815108224-update-data-sharing-ingestion-capabilities.js
Outdated
Show resolved
Hide resolved
opencti-platform/opencti-graphql/src/modules/vocabulary/vocabulary.graphql
Outdated
Show resolved
Hide resolved
Question: When I check the capability to "access" data sharing but not the "manage" data sharing box I cannot access to the data sharing menu at all. Is it the attended behavior ? |
0d207c0
to
b0a1ea1
Compare
It's true, I naively thought that the existing state of the ‘data sharing’ section was correct. => fixed |
b0a1ea1
to
0b9b5e0
Compare
It's ready for a 2nd round of review |
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.
Retested ✅
Tested locally, I cannot import a csv file without having settings capa, but not sure if it's related to this PR or not. |
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.
please rename migration to update its timestamp before merge
...m/opencti-graphql/src/migrations/1716815108224-update-data-sharing-ingestion-capabilities.js
Outdated
Show resolved
Hide resolved
6ed57f2
to
cb3944e
Compare
cb3944e
to
609fbae
Compare
...m/opencti-graphql/src/migrations/1718263084981-update-data-sharing-ingestion-capabilities.js
Outdated
Show resolved
Hide resolved
When having "access ingestion" capabilility (without manage ingestion), I can see the edit buttons of CSV feeds and if I try to edit it redirects me to login page: I also have the button + to create a new CSV ingestion but it also redirects me to login page. I guess it's because I'm trying to load CSV mappers list and it's protected by INGESTION_SETCSVMAPPERS capability For RSS feeds and Taxii feeds, I only see the list, without any action => DONE |
d850a9a
to
7bf7e33
Compare
opencti-platform/opencti-front/src/private/components/data/Root.jsx
Outdated
Show resolved
Hide resolved
7927b1b
to
d86d713
Compare
dfc368c
to
a8dc7b9
Compare
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.
make sure before merging that the migration timestamp is up to date with master
Proposed changes
MAJOR :
MINOR :
Related issues
Checklist
Further comments
Capabilities properties :
Access Ingestion
Manage Ingestion