Don't think we need to tag the two low level platforms as a separate station.

I have reverted this changeset as trains still stop here until 9 December.

Presumably this is meant to be tagged as a highway of some kind? way/626844277
And surely the ways on either side are paths rather than tracks, given the steps?

Ok, thanks. I added these as under construction based on twitter images I saw on @SCWFriends, so I've set them as open now. If you know exactly where they end feel free to update. Be good to also have the speed limit on this and surrounding roads :)

Have you managed to survey this and if so, are these cycle tracks fully open?

Hi, noticed you added a road - way/643551290 - however it was only connected to the cycle path. I've fixed it and connected it to the roundabout instead.

Not really much point in adding a duplicate residential area when the town is already covered by one...

Hi, welcome to OSM and thanks for the edit. I've fixed this house so that the house number and street are separate pieces of data. Any questions feel free to ask.

Hi, welcome to OSM and thanks for adding the cycle path.
I've edited it so that it connects to the other paths and side roads. :)

Hi, please be careful when mapping as you managed to drag a road junction across the map :) https://overpass-api.de/achavi/?changeset=64117996 - I've fixed it.

way/384765377 removed the access=no on part of this roundabout, assuming it was just for the FRB closure - revert if not

I was just thinking about routers, as effectively it's a dead end.
Doesn't look like it's on the core paths plans.

I think I did this copying another one which I saw mapped as a way. A shame that the site relation name is not rendered though. :(

Is the HArTT route roundtrip=yes or is there supposed to be a gap in your relation? http://ra.osmsurround.org/analyzeMap?relationId=8476048

Hi and welcome to OSM - thanks for the correction :)

Presumably one or more of the paths at the north end allow cycle access?

I've changed node/469415682 back to motorway_junction as the wiki mentions it can be used with trunk roads.

Hi, I've changed this to railway=junction node/5268885998

I think way/635792335 would be better tagged as a breakwater?

Thanks for adding the cycle path, saw construction photos on fb. I've marked it as lcn=yes to complete the route to Kilrenny. Did you notice if there was a path alongside the road here? node/908836830