Freegle / Iznik / 22372

71%

master: 71%

Build Type

Pull #618

circleci

Committed by edwh

Commit Message

feat(bulk-offer): never send bulk/clearance offers to LoveJunk

LoveJunk's draft model is one item per post and can't represent a
multi-item clearance, so bulk offers (messages with messages_bulk_items
rows) must not be pushed at all. Add a NOT EXISTS guard to the
"new offers" selection in both the live Laravel sync (LoveJunkService)
and the legacy V1 cron (iznik-server/scripts/cron/lovejunk.php). Because
bulk offers then never enter the lovejunk table, the edit/outcome paths
skip them too.

Test: test_skips_bulk_offer_message asserts a message with bulk items is
neither sent nor recorded. Full LoveJunkService suite green (13).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015f3YY7RTNdsW6WSEa3dRWg

Pull Request Pull Request #618: feat(bulk-offer): structured multi-item clearance listings + per-item interest

Coverage Stats

11296 of 15014 branches covered (75.24%)

Branch coverage included in aggregate %.

2080 of 2593 new or added lines in 23 files covered. (80.22%)

2 existing lines in 1 file now uncovered.

120965 of 171406 relevant lines covered (70.57%)

36.05 hits per line