-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
tests Binding to List test #2823
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
Conversation
Added BindList for use with SourceList
|
An issue with this one is SourceList is not meant to be exposed to the UI layer at all. Its fundamnetally against the way DD works. |
|
Why use a SourceList directly is the other concern I have, it's meant to be for data which you observe, transform, filter etc. |
Codecov Report
@@ Coverage Diff @@
## main #2823 +/- ##
==========================================
+ Coverage 68.26% 68.28% +0.02%
==========================================
Files 137 137
Lines 4787 4787
==========================================
+ Hits 3268 3269 +1
+ Misses 1519 1518 -1
Continue to review full report at Codecov.
|
|
Ok, I will rethink this to try a different approach |
|
0.02% hmmm every little helps I guess |
|
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Added Bind to List test with ReadOnlyObservableCollection
put solution items into two folders, Src and Tests
What kind of change does this PR introduce?
test
What is the current behaviour?
What is the new behaviour?
added test
What might this PR break?
none expected
Please check if the PR fulfils these requirements
Other information: