Skip to content

Conversation

@jacobaraujo7
Copy link
Contributor

@jacobaraujo7 jacobaraujo7 commented Dec 10, 2021

GOAL:

  • Methods similar to Provider to facilitate possible migration between the two packages. For this, it will be necessary to add the reading of binds also via BuildContext [context.read()] and watch the changes [context.watch()];

  • Add context.read() method by extension.

  • Create InheritedWidget for register by context.dependOnInheritedWidgetOfExactType.

  • Add context.watch() method by extension.

  • Add tests

  • Add Dartdocs

  • Update official documentation

@codecov
Copy link

codecov bot commented Dec 10, 2021

Codecov Report

Merging #617 (41d89ca) into master (3e5b530) will decrease coverage by 5.25%.
The diff coverage is 19.23%.

❗ Current head 41d89ca differs from pull request most recent head a6b85cb. Consider uploading reports for the commit a6b85cb to get more accurate results
Impacted file tree graph

@@             Coverage Diff             @@
##            master     #617      +/-   ##
===========================================
- Coverage   100.00%   94.74%   -5.26%     
===========================================
  Files           32       32              
  Lines          751      799      +48     
===========================================
+ Hits           751      757       +6     
- Misses           0       42      +42     
Impacted Files Coverage Δ
...modular/lib/src/presenter/widgets/modular_app.dart 40.84% <19.23%> (-59.16%) ⬇️

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 30a32f3...a6b85cb. Read the comment docs.

@jacobaraujo7 jacobaraujo7 merged commit bd0fdaf into master Dec 10, 2021
@jacobaraujo7 jacobaraujo7 deleted the add-watchbinds branch December 10, 2021 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants