mobx.dart icon indicating copy to clipboard operation
mobx.dart copied to clipboard

mobx utils

Open altynbek132 opened this issue 2 years ago • 5 comments

added mobx utils

  • obs from Listenable
  • obs from ValueNotifier
  • Stream of obs values

at least we should have ability to convert obs values into stream since streams are very useful for async world https://github.com/mobxjs/mobx-utils#tostream for example we could convert computed field to stream and solve #860

added reporting manual change for observable and collections there are some cases when you should need to report change, for example if using collections extensions (sortBy), or writing own function that converts some class similar to Listenable.

altynbek132 avatar Dec 12 '22 20:12 altynbek132

Codecov Report

Merging #886 (60a3085) into master (13b7e6d) will decrease coverage by 0.77%. The diff coverage is 0.00%.

:exclamation: Current head 60a3085 differs from pull request most recent head 7193902. Consider uploading reports for the commit 7193902 to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #886      +/-   ##
==========================================
- Coverage   98.96%   98.19%   -0.77%     
==========================================
  Files          55       55              
  Lines        1924     1939      +15     
==========================================
  Hits         1904     1904              
- Misses         20       35      +15     
Flag Coverage Δ
flutter_mobx 100.00% <ø> (ø)
mobx 97.46% <0.00%> (-1.08%) :arrow_down:
mobx_codegen 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...rc/api/observable_collections/observable_list.dart 98.76% <0.00%> (-1.24%) :arrow_down:
...src/api/observable_collections/observable_map.dart 97.45% <0.00%> (-2.55%) :arrow_down:
...src/api/observable_collections/observable_set.dart 95.23% <0.00%> (-4.77%) :arrow_down:
mobx/lib/src/core/observable.dart 89.13% <0.00%> (-10.87%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 13b7e6d...7193902. Read the comment docs.

codecov[bot] avatar Dec 12 '22 23:12 codecov[bot]

This looks very useful too. Anyone else have review and merge perms?

ghenry avatar Dec 19 '22 11:12 ghenry

@pavanpodila

altynbek132 avatar Dec 19 '22 12:12 altynbek132

I also implemented some features of mobx-utils. https://github.com/amondnet/mobx_utils

amondnet avatar Dec 19 '22 16:12 amondnet

Coverage has dropped. Can we fix that @amondnet ?

pavanpodila avatar Dec 20 '22 01:12 pavanpodila