valitydev / hooker

Apache License 2.0
0 stars 0 forks source link

Add time check #63

Closed strug closed 1 month ago

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 96.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 64.47%. Comparing base (02a4309) to head (f7a3e5c). Report is 22 commits behind head on master.

Files with missing lines Patch % Lines
...a/dev/vality/hooker/dao/impl/InvoicingDaoImpl.java 94.44% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #63 +/- ## ============================================ + Coverage 61.99% 64.47% +2.48% - Complexity 307 348 +41 ============================================ Files 68 69 +1 Lines 1563 1720 +157 Branches 124 141 +17 ============================================ + Hits 969 1109 +140 - Misses 529 549 +20 + Partials 65 62 -3 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

strug commented 1 month ago

https://github.com/empayre/inner-docs/blob/dddae618f63914c35c40cadce3dfddbc39bb247f/docs/arch/rfc/techdept/hooks/001-problem-with-parent-hook.md

strug commented 1 month ago

У меня возник вопрос. После создания нового хука( hook_2) события, которые были в первом хуке(hook_1) типа event_1, уже не придут в новом хуке ? То есть будет уже новая последовательность событий в рамках инвойса, которая была сгенерирована после создания hook_2? Верно?

Да не придут тк. гарантий что придет вся история нет, есть только что последовательность будет сохранена в разрезе выполняемых событий