Skip to content

Adding new London Overground line names#33

Open
tgraynz wants to merge 3 commits into
benjamintd:mainfrom
tgraynz:TG/london-overground
Open

Adding new London Overground line names#33
tgraynz wants to merge 3 commits into
benjamintd:mainfrom
tgraynz:TG/london-overground

Conversation

@tgraynz

@tgraynz tgraynz commented Jan 7, 2026

Copy link
Copy Markdown

Added a custom preprocess script for London, manually changed source.json to create the new London Overground lines. Unable to fully test due to mapbox style issues.

@vercel

vercel Bot commented Jan 7, 2026

Copy link
Copy Markdown
Contributor

@tgraynz is attempting to deploy a commit to the Benjamin Td Team on Vercel.

A member of the Team first needs to authorize it.

@vercel

vercel Bot commented Jan 8, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
metro-memory-com Error Error May 26, 2026 9:31am

Request Review

@tgraynz

tgraynz commented Jan 8, 2026

Copy link
Copy Markdown
Author

I've tested the Preview, and functionality for the new overground lines looks good to me. The route lines are still showing as the old colour on the map, which I've traced to a setting in the London 'config.ts' - if I set MAP_FROM_DATA to true then I see the correct line colours on the map. I don't know the history for why this setting exists, or how the lines are drawn if this isn't set, so I'm not entirely confident in adding that setting in - it may be preferable to change the line colours within the current method for drawing the lines.

One thing that has occurred to me - I have preserved station id's for all other lines, and assigned new station id's for all stations on the new overground lines. This means that previous game scores will be slightly impacted, because for example if they had found Richmond, this previously would have counted as 2 (one for the District line and one for the Overground), whereas now only the District line station will be 'found' and the new Mildmay line station id will not be found, so they would have to enter Richmond again to go back to the previous state. Not sure how big a deal this is - there are options for working around this, but I can't think of a clean solution, particularly because there are stations that have multiple overground lines as well as tube lines, and so the overall station count has gone up.

@benjamintd

Copy link
Copy Markdown
Owner

Historically I would make a separate style in my Mapbox account for each city, e.g. to be able to tweak the line positions/offsets (two overlapping lines would receive offsets to appear parallel to each other).

Example with the yellow and green lines:
image

Since then, I realized it was easier to load the geojson from the code and display that on the map as geojson, and I added the MAP_FROM_DATA configuration.

Since the most accurate data now lives in your geojson file I'd be okay to use it directly instead?

Perhaps an improvement needed would be to allow offsets from the map data directly, from a feature property... That might be outside the scope of this PR but you're welcome to take a shot at it!

@tgraynz

tgraynz commented Jan 8, 2026

Copy link
Copy Markdown
Author

Ok that makes sense, yeah agreed may as well use the geojson file for London now. I will commit that change then.

I did wonder about playing around with line styles - it would be good to have the parallel line style for the overground lines and the Elizabeth line, particularly because now the Windrush line looks very similar to the Central line. And that would require basically the same feature - offsetting orthogonally. But yes outside the scope of this PR.

@benjamintd

Copy link
Copy Markdown
Owner

would you be interested in contributing this change? In this or another PR.

@tgraynz

tgraynz commented Jan 8, 2026

Copy link
Copy Markdown
Author

Yeah for sure, I have a few other ideas for features I'd like to play around with as well

@tgraynz

tgraynz commented Jan 23, 2026

Copy link
Copy Markdown
Author

I'm a little bit stuck here - the preview is still showing the old orange colour for all the overground lines, but my local testing shows the correct colours. Any ideas as to what could be going on?

@benjamintd

Copy link
Copy Markdown
Owner
Error occurred prerendering page "/". Read more: https://nextjs.org/docs/messages/prerender-error
TypeError: r is not iterable
    at E (/vercel/path0/.next/server/app/(website)/page.js:9:14294)
    at N (/vercel/path0/.next/server/app/(website)/page.js:9:13446)
    at D (/vercel/path0/.next/server/app/(website)/page.js:9:14769)
    at er (/vercel/path0/.next/server/app/(website)/page.js:9:22008)
    at em (/vercel/path0/node_modules/next/dist/compiled/next-server/app-page.runtime.prod.js:12:128381)
    at /vercel/path0/node_modules/next/dist/compiled/next-server/app-page.runtime.prod.js:12:140081
    at Object.toJSON (/vercel/path0/node_modules/next/dist/compiled/next-server/app-page.runtime.prod.js:12:143659)
    at stringify (<anonymous>)
    at eE (/vercel/path0/node_modules/next/dist/compiled/next-server/app-page.runtime.prod.js:12:132044)
    at eR (/vercel/path0/node_modules/next/dist/compiled/next-server/app-page.runtime.prod.js:12:132487)

 ✓ Generating static pages (77/77) 
> Export encountered errors on following paths:
	/(website)/page: /
Error: Command "npm run build" exited with 1

I'm getting this error on deploy; i'll take a look when I have time, sorry !

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