chore: remove debug console.log from metadataReducer.ts #452
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
- Replace static hex maps with interactive choropleth maps - Use actual constituency boundaries from GeoJSON - Hover shows constituency name, % households affected, properties, revenue - Maps embedded via iframe for performance 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
- Add revenue concentration tables (top constituencies for 50%/75%/90%) - Add London dominance statistics - Add distribution breakdown by number of affected properties - Add hideHeaderImage option to BlogPost type for posts that should hide cover image on article page while still showing in cards - Reduce header padding when image is hidden 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
- Create merged CSV with properties and revenue for both thresholds - Add download link in methodology section 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
Matches the style of UK Autumn Budget Dashboard: - D3.js with actual GeoJSON constituency boundaries - British National Grid projection - Sequential teal color scale - Click-to-zoom with tooltips - Search functionality - Zoom controls 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
- Changed 0% fill color from #ddd to #f5f5f5 (lighter) - All constituencies now show tooltips on click, including those with 0 properties 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
- Changed 0% fill to light blue-gray (#e0e7ed) for better contrast - White background on map canvas 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
- Toggle between geographic boundaries and hex cartogram views - Hex view gives equal visual weight to each constituency - Makes London constituencies (small but high impact) more visible - Smooth animated transitions between views - Search and zoom work in both views 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
Temporary fix for social cards - copied post entry to: - app/api/posts.json - app/public/data/posts.json 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
- Remove "% households affected" since we don't know who owns properties - Color scale now based on number of properties - Simplified tooltip: just Properties and Est. revenue - Numbers aligned to right side in tooltips - Updated tables and methodology text 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
- Change color scale from number of properties to % of constituency - Update legend to show "Lower %" / "Higher %" - Keep tooltips showing both properties count and percentage 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
- Map tooltips: shortened to "Number" and "Percent" - Blog tables: full "Number of properties" and "Percent of properties" - Search results: show count and percentage without extra text 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
- Replace obscure aggregator links with FT primary source - Clarify "Percent of properties" → "Share of total" to indicate share of all affected properties, not share of constituency - Round percentages to 1 decimal place for cleaner display 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
Reran analysis with OBR HPI uprating (1.0356x to 2026-27 values): - £1.5m: 13,884 → 14,820 properties (+6.7%) - £2m: 7,849 → 8,213 properties (+4.6%) Updated all tables and statistics to reflect uprated analysis. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
- Reframe as geographic proxy (sales data, not stock) - Fix source attribution (The Times, not FT) - Add cover image from hex map screenshot - Streamline prose: remove passive voice, redundant tables - Add key stats: top 3 concentration, median, outside London leaders - Add external CSS for map sizing 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
- Combined two maps into single interactive with £1.5m/£2m toggle - Removed fictional £2,000 revenue estimates (unsourced) - Changed "affected properties" to "sales" throughout - Updated title to "Estimating the constituency distribution..." - Added postcode-to-constituency methodology detail - Table now shows share of nationwide high-value sales - Removed revenue from hover tooltips 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
- Add mansion tax as 4th card in Pre-autumn budget analysis section - Reduce card widths from 320px to 260px for better fit with 4 cards 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <[email protected]>
Collaborator
|
Please rebase to remove the portions of this PR that are Daphne's mansion tax article. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
console.logfrom parameter tree buildconsole.errorfor legitimate failure case with eslint-disable commentTest plan
🤖 Generated with Claude Code