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

Implemented change notification when a table changes by the query. #733

Merged

Conversation

dkaera
Copy link
Collaborator

@dkaera dkaera commented Feb 11, 2023

Resolves issue #731

@codecov
Copy link

codecov bot commented Feb 11, 2023

Codecov Report

Merging #733 (0f6564f) into develop (6e507dc) will increase coverage by 1.15%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #733      +/-   ##
===========================================
+ Coverage    90.76%   91.92%   +1.15%     
===========================================
  Files           11       11              
  Lines          195      223      +28     
===========================================
+ Hits           177      205      +28     
  Misses          18       18              
Flag Coverage Δ
floor 91.92% <100.00%> (+1.15%) ⬆️

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

Impacted Files Coverage Δ
floor/lib/src/adapter/query_adapter.dart 97.14% <100.00%> (+1.90%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@dkaera dkaera mentioned this pull request Feb 11, 2023
@dkaera dkaera force-pushed the feature/notify_on_query_modify_table branch from afbdd07 to f78c6b6 Compare February 11, 2023 18:25
@dkaera dkaera force-pushed the feature/notify_on_query_modify_table branch from aca7147 to a3bcff6 Compare February 11, 2023 21:19
@dkaera dkaera force-pushed the feature/notify_on_query_modify_table branch from a3bcff6 to aee2e18 Compare February 11, 2023 21:24
memoriasIT
memoriasIT previously approved these changes Mar 7, 2023
ThomasMiddel
ThomasMiddel previously approved these changes Apr 18, 2023
@dkaera dkaera dismissed stale reviews from ThomasMiddel and memoriasIT via 6bbdf29 April 18, 2023 11:08
@ThomasMiddel ThomasMiddel merged commit f465085 into pinchbv:develop Apr 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants