fix(spend_tracking): /spend/logs
with no filter
#4998
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Title
Fix
/spend/logs
"cannot access local variable 'spend_log' where it is not associated with a value" error caused by a typo in implementation.Relevant issues
No issue. But the bug does exist.
Steps to reproduce the bug
curl localhost:4000/spend/logs -H 'Authorization: Bearer <key here>'
Expected output
Spending logs
Actual output
Type
🐛 Bug Fix
Changes
spend_logs
-->spend_log
in the handling logic when no filter is setspend_logs
at greater scope[REQUIRED] Testing - Attach a screenshot of any new tests passing locall
curl localhost:4000/spend/logs -H 'Authorization: Bearer <key here>'
can now work properly.All failed tests are failing before change.