Skip to content

Conversation

@bokelley
Copy link
Contributor

@bokelley bokelley commented Nov 6, 2025

Summary

  • Replace dynamic ad unit search with button-triggered search to eliminate input lag
  • Invalidate inventory cache after successful sync to show fresh data
  • Add last_sync timestamp to inventory tree API response
  • Fix ad unit sizes display in inventory details panel

Test plan

  • Sync inventory and verify "Last synced" timestamp shows (not "Never synced")
  • Type in ad unit search box and confirm no automatic searches occur
  • Click Search button or press Enter to trigger search
  • Click an ad unit in inventory browser and verify sizes display in details panel

- Replace dynamic ad unit search with button-triggered search to eliminate lag
- Invalidate inventory cache after successful sync for fresh data display
- Add last_sync timestamp to inventory tree API response
- Fix ad unit sizes display in inventory details panel

This addresses three issues: search input lag, stale cached inventory after sync, and missing ad unit sizes in the details view.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@bokelley bokelley merged commit 0d09f1b into main Nov 6, 2025
10 checks passed
danf-newton pushed a commit to Newton-Research-Inc/salesagent that referenced this pull request Nov 24, 2025
…ol#709)

- Replace dynamic ad unit search with button-triggered search to eliminate lag
- Invalidate inventory cache after successful sync for fresh data display
- Add last_sync timestamp to inventory tree API response
- Fix ad unit sizes display in inventory details panel

This addresses three issues: search input lag, stale cached inventory after sync, and missing ad unit sizes in the details view.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants