fix: lottery history log filtering

emilyjablonski commented 2 months ago

This PR addresses #4253

[x] Addresses the issue in full
[ ] Addresses only certain aspects of the issue

Description

Lottery history logs are not currently filtering by ID. The issue was that the listing ID wasn't actually getting passed through to the endpoint and was coming in as undefined.

How Can This Be Tested/Reviewed?

Create two listings and take lottery actions on each of them. Ensure their history logs only reflect the actions that were taken on that listing.

Author Checklist:

[x] Added QA notes to the issue with applicable URLs
[ ] Reviewed in a desktop view
[ ] Reviewed in a mobile view
[ ] Reviewed considering accessibility
[ ] Added tests covering the changes
[ ] Made corresponding changes to the documentation
[ ] Ran yarn generate:client and/or created a migration when required

Review Process:

Read and understand the issue
Ensure the author has added QA notes
Review the code itself from a style point of view
Pull the changes down locally and test that the acceptance criteria is met
Either (1) explicitly ask a clarifying question, (2) request changes, or (3) approve the PR, even if there are very small remaining changes, if you don't need to re-review after the updates

netlify[bot] commented 2 months ago

Deploy Preview for partners-bloom-dev ready!

Name	Link
Latest commit	9cb9403bd9235040377a7f232a1358915fea0cb5
Latest deploy log	https://app.netlify.com/sites/partners-bloom-dev/deploys/66be704dc18f2300084845b4
Deploy Preview	https://deploy-preview-4254--partners-bloom-dev.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

netlify[bot] commented 2 months ago

Deploy Preview for bloom-exygy-dev ready!

Name	Link
Latest commit	9cb9403bd9235040377a7f232a1358915fea0cb5
Latest deploy log	https://app.netlify.com/sites/bloom-exygy-dev/deploys/66be704d5d884d0008d10746
Deploy Preview	https://deploy-preview-4254--bloom-exygy-dev.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

bloom-housing / bloom