-
Notifications
You must be signed in to change notification settings - Fork 799
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] extends Audits context filters to Knowledge events (#5611) #5613
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Archidoit
added
filigran team
use to identify PR from the Filigran team
filters
temporarily used to identify filter issue to facilitate filter team’s work identification
labels
Jan 19, 2024
Archidoit
changed the title
[backend] refacto completeContextDataForEntity (#5611)
[backend] fix context filters in Audits (#5611)
Jan 19, 2024
Archidoit
changed the title
[backend] fix context filters in Audits (#5611)
[backend] extends Audits context filters to Knowledge (#5611)
Jan 19, 2024
ready for review, but need check with Julien before merge |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #5613 +/- ##
==========================================
+ Coverage 65.68% 65.71% +0.03%
==========================================
Files 539 539
Lines 63570 63482 -88
Branches 5128 5131 +3
==========================================
- Hits 41754 41718 -36
+ Misses 21816 21764 -52 ☔ View full report in Codecov by Sentry. |
Archidoit
changed the title
[backend] extends Audits context filters to Knowledge (#5611)
[backend] extends Audits context filters to Knowledge events (#5611)
Jan 22, 2024
Archidoit
force-pushed
the
issue/5611
branch
2 times, most recently
from
January 31, 2024 14:30
5060045
to
dec215d
Compare
SamuelHassine
force-pushed
the
master
branch
4 times, most recently
from
February 4, 2024 22:22
3972324
to
3d01224
Compare
jpkha
reviewed
Feb 5, 2024
LGTM, I am letting @richard-julien to finalize the review |
SamuelHassine
force-pushed
the
master
branch
2 times, most recently
from
February 9, 2024 01:54
6f29a12
to
ebfecd0
Compare
Archidoit
force-pushed
the
issue/5611
branch
from
February 16, 2024 15:10
3b17ce4
to
0f5e664
Compare
richard-julien
force-pushed
the
issue/5611
branch
from
February 17, 2024 10:51
a8e88f9
to
9ee3e5a
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
filigran team
use to identify PR from the Filigran team
filters
temporarily used to identify filter issue to facilitate filter team’s work identification
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
issue: #5611