Closed ozum closed 6 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 100.00%. Comparing base (
ebb6c01
) to head (15c64e7
). Report is 43 commits behind head on main.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Thank you @ozum for making this important contribution! π I released your fix with trading-signals v5.0.1 βοΈ
The replace feature in the
update
of theTR
indicator was not working correctly, because it does not handlepreviousCandle
when replacing the result.I added a new attribute to store two previous candles that replace the previous candle when
replace
istrue
. I also included a test to ensure that replace feature is working as expected.