Hmm, the SR was probably national because the Scottish part is national, maybe it should be two separate relations (without an SR) for either side of the border?

What kind of shop is this? As the name has been changed but not the shop= tag.

I'm wondering if there's a need for this super relation?

There's six caravan sites tagged here way/949960395 - these areas should probably be merged into one, or even merged with the campsite to avoid duplication?

whops, I'd added a tennis court and must've clicked on the wrong preset next thing I added - fixed.

There's already a Leith Fort node/3966284094/history though it was changed to a locality (not sure that's correct as it's a populated place).

Why have you marked way/907680656/history as access=no?

reverted

Thanks for adding these :) Only thing I'm wondering if whether they should be tagged as route=mtb instead?

Are the schools open? If so amenity=school should be better added to them.

Not sure why you made this change as the line is not a path, but I've fixed it.

No need for a highway=services area here, as this is just a petrol station.

Hi, is there a particular reason why you've merged various POI nodes and buildings together in Edinburgh?

Did you mean to remove the postcode?

Thanks, assume this can be resolved? note/3026713

Is way/827503886 signed as a cycle path?

Presumably it will be a continuous pavement as the plans show it will be one way https://www.edinburgh.gov.uk/ccwel/downloads/download/8/general-arrangements

Fairly sure this is being kept but changed to one way...

thanks but it was already marked as railway=abandoned so I've readded it

See note: note/3302991