Skip to content
This repository has been archived by the owner on May 20, 2024. It is now read-only.

Rename feedback_possible field to is_done #642

Merged
merged 5 commits into from
Feb 10, 2019
Merged

Conversation

nicksellen
Copy link
Member

This is part of karrot-dev/karrot-frontend#709 - with the idea that if pickups had a long duration, they might be done part way through, and so would be useful to be able to mark them as done.

This doesn't add the API to mark them as done. Yet.

Concept of "feedback possible" still exists seperately
@codecov
Copy link

codecov bot commented Jan 29, 2019

Codecov Report

Merging #642 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #642      +/-   ##
=========================================
+ Coverage    95.8%   95.8%   +<.01%     
=========================================
  Files         362     363       +1     
  Lines       13970   13974       +4     
  Branches      858     858              
=========================================
+ Hits        13384   13388       +4     
  Misses        468     468              
  Partials      118     118
Impacted Files Coverage Δ
foodsaving/users/stats.py 100% <ø> (ø) ⬆️
foodsaving/stores/tests/test_stores_api.py 100% <ø> (ø) ⬆️
foodsaving/pickups/models.py 95.75% <100%> (ø) ⬆️
...ving/pickups/migrations/0014_auto_20190129_1843.py 100% <100%> (ø)
foodsaving/notifications/tests/test_receivers.py 100% <100%> (ø) ⬆️
foodsaving/pickups/tests/test_models.py 100% <100%> (ø) ⬆️
foodsaving/subscriptions/receivers.py 96.61% <100%> (ø) ⬆️
foodsaving/notifications/receivers.py 97.54% <100%> (ø) ⬆️

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 083e29f...61c2053. Read the comment docs.

1 similar comment
@codecov
Copy link

codecov bot commented Jan 29, 2019

Codecov Report

Merging #642 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #642      +/-   ##
=========================================
+ Coverage    95.8%   95.8%   +<.01%     
=========================================
  Files         362     363       +1     
  Lines       13970   13974       +4     
  Branches      858     858              
=========================================
+ Hits        13384   13388       +4     
  Misses        468     468              
  Partials      118     118
Impacted Files Coverage Δ
foodsaving/users/stats.py 100% <ø> (ø) ⬆️
foodsaving/stores/tests/test_stores_api.py 100% <ø> (ø) ⬆️
foodsaving/pickups/models.py 95.75% <100%> (ø) ⬆️
...ving/pickups/migrations/0014_auto_20190129_1843.py 100% <100%> (ø)
foodsaving/notifications/tests/test_receivers.py 100% <100%> (ø) ⬆️
foodsaving/pickups/tests/test_models.py 100% <100%> (ø) ⬆️
foodsaving/subscriptions/receivers.py 96.61% <100%> (ø) ⬆️
foodsaving/notifications/receivers.py 97.54% <100%> (ø) ⬆️

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 083e29f...61c2053. Read the comment docs.

@codecov
Copy link

codecov bot commented Jan 29, 2019

Codecov Report

Merging #642 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #642      +/-   ##
=========================================
+ Coverage   94.99%     95%   +<.01%     
=========================================
  Files         368     369       +1     
  Lines       14338   14349      +11     
  Branches      895     895              
=========================================
+ Hits        13621   13632      +11     
  Misses        591     591              
  Partials      126     126
Impacted Files Coverage Δ
...saving/pickups/tests/test_pickupdate_series_api.py 99.44% <ø> (ø) ⬆️
foodsaving/pickups/serializers.py 96.42% <ø> (ø) ⬆️
foodsaving/places/tests/test_stores_api.py 100% <ø> (ø) ⬆️
foodsaving/pickups/models.py 92.52% <100%> (ø) ⬆️
foodsaving/notifications/tests/test_receivers.py 100% <100%> (ø) ⬆️
foodsaving/pickups/tests/test_pickupdates_api.py 99.33% <100%> (+0.01%) ⬆️
foodsaving/subscriptions/receivers.py 96.61% <100%> (ø) ⬆️
foodsaving/pickups/tests/test_models.py 100% <100%> (ø) ⬆️
...ving/pickups/migrations/0015_auto_20190129_1843.py 100% <100%> (ø)
foodsaving/notifications/receivers.py 97.58% <100%> (ø) ⬆️
... and 1 more

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 749ef1a...15b419e. Read the comment docs.

@nicksellen nicksellen changed the title WIP: Rename feedback_possible field to is_done Rename feedback_possible field to is_done Feb 9, 2019
@nicksellen
Copy link
Member Author

Rebase/renamed so it fits on top of master now.

@tiltec tiltec merged commit e893fdc into master Feb 10, 2019
@tiltec tiltec deleted the add-pickup-done-field branch February 10, 2019 11:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants