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

Removed duplicated ApplyDataFilters usage for MongoDbRepository.FindAsync #17641

Merged
merged 1 commit into from Sep 15, 2023

Conversation

hikalkan
Copy link
Member

Removed ApplyDataFilters call for the FindAsync method of mongodb repository. Because the GetMongoQueryableAsync method already calls it.

Can you double-check if there is a problem on that?

@codecov
Copy link

codecov bot commented Sep 14, 2023

Codecov Report

Merging #17641 (4572bbd) into dev (1eb01ee) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##              dev   #17641   +/-   ##
=======================================
  Coverage   53.51%   53.51%           
=======================================
  Files        3035     3035           
  Lines       94848    94850    +2     
=======================================
+ Hits        50760    50763    +3     
+ Misses      44088    44087    -1     
Files Changed Coverage Δ
...p/Domain/Repositories/MongoDB/MongoDbRepository.cs 77.90% <100.00%> (+0.08%) ⬆️

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@realLiangshiwei realLiangshiwei merged commit caf98f1 into dev Sep 15, 2023
5 checks passed
@realLiangshiwei realLiangshiwei deleted the remove-duplicate-applyfilters branch September 15, 2023 01:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants