search

tsbonev / nharker

NHaker - an offline writing organization tool
0 stars 0 forks source link

articles: fix property detaching #176

Closed tsbonev closed 6 years ago

tsbonev commented 6 years ago

Detaching a property now returns the updated article and the id of the dereferenced property, making it possible to spawn a DeleteEntryCommand from the workflow. This completes the last point of- and closes #159. This also fixes #175.

codecov[bot] commented 6 years ago

Codecov Report

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

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #176      +/-   ##
============================================
+ Coverage     96.77%   96.78%   +<.01%     
  Complexity      278      278              
============================================
  Files            31       31              
  Lines           869      870       +1     
  Branches         79       79              
============================================
+ Hits            841      842       +1     
  Misses            5        5              
  Partials         23       23
Impacted Files Coverage Δ Complexity Δ
...tsbonev/nharker/adapter/nitrite/NitriteArticles.kt 100% <100%> (ø) 34 <0> (ø) :arrow_down:
...tsbonev/nharker/server/workflow/ArticleWorkflow.kt 96.89% <100%> (+0.02%) 29 <0> (ø) :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 4d96bec...396f657. Read the comment docs.