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

[backend/frontend] entity_type filter available for abstract types #6064

Merged
merged 2 commits into from
Feb 22, 2024

Conversation

Archidoit
Copy link
Member

Issue (on testing only) : entity_type filter not available in Data>Entities, Observations>Observables, Global Search

@Archidoit 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 Feb 21, 2024
@Archidoit Archidoit self-assigned this Feb 21, 2024
@Archidoit Archidoit marked this pull request as draft February 21, 2024 10:05
Copy link

codecov bot commented Feb 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (62f3386) 65.94% compared to head (02e2420) 65.95%.
Report is 4 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #6064   +/-   ##
=======================================
  Coverage   65.94%   65.95%           
=======================================
  Files         541      541           
  Lines       64286    64289    +3     
  Branches     5199     5202    +3     
=======================================
+ Hits        42395    42403    +8     
+ Misses      21891    21886    -5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Archidoit Archidoit marked this pull request as ready for review February 21, 2024 10:46
@Kedae
Copy link
Member

Kedae commented Feb 21, 2024

Can we have a small e2e test for that ? :)

@Archidoit
Copy link
Member Author

Archidoit commented Feb 21, 2024

@Kedae I added a backend test since the problem is only from the back, but I can also add an e2e :)

@Archidoit Archidoit force-pushed the issue/4949-2 branch 2 times, most recently from 59035d5 to 58f6849 Compare February 21, 2024 15:15
Copy link
Contributor

@JeremyCloarec JeremyCloarec left a comment

Choose a reason for hiding this comment

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

Tested OK

@Archidoit Archidoit merged commit fee8c4f into master Feb 22, 2024
8 checks passed
@Archidoit Archidoit deleted the issue/4949-2 branch February 22, 2024 09:59
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants