Those fixes both look good.
There is a restriction at the north end already mapped, restricting to only a left turn.
relation/7382486/

Hi, thanks for adding these details, they look good to me.

For opening hours, if you're unsure then it's best to use something like this to check them: https://projets.pavie.info/yohours/?oh=Mo%2008:00-18:00;%20Tu-Fr%2008:00-20:00;%20Sa%2008:00-19:00;%20Su%2009:00-16:00

I've put in the opening hours as I've fixed them there, so double check that that's what you meant.

I've fixed it for you here, and on Moxon's next door too.
Looks like you were close, it's just it should be two letter days, not three letters, and you missed a hyphen.

Hi, you requested a review of this change.
Thanks for updating this.
I can see your problem with not knowing how to tag it. What is it used for when not by circuses or fairs?

Hi, you requested a review of this change.
On the crossing: node/3892644/ you've changed them from traffic signals to marked crossing. If there are no traffic lights then you should also remove the crossing_ref tag (or if it's something like a zebra crossing, then change it appropriately).
Is Nightingale Walk one way for the whole length, or is the small section at the south end not oneway?
Other than that, looks good and looks like you've added good details here.

Hi and welcome to OpenStreetMap. You requested a review of this change.
Thanks for adding all these paths, always good to have this sort of detail, a few comments and questions though.
Here node/8432600246/ and just next to it, you've put crossings in, the tags you've used are for where roads and paths cross each other though. Perhaps they are bridges or fords?
Some of the names you've added seem more like descriptions (you can change the tag from "name" to "description" if that's the case. The name should only be used for the names as they are signed on the ground. Also, I think it should always be "Tormore Forest", not "Tormore forest".
On this path: way/908149866 you'd added that it was a ford for the whole length, presumably there are one or more points where the fords are.
Also, make sure that you connect paths up to the roads that they end on, to enable them to be routed along.
I've been in and connected up the paths, removed that ford tag and added one where it looks like it might be.
Finally, you've marked the entrance: node/8432600242 but tagged it as a park. If there's a sign there, then tag that point as a sign, otherwise that's not how that tag should be used, so the point should be removed.

For car park, just search that and there is a feature for it (either as a point or as an area).
Same for viewpoint, and for seat try bench, or maybe picnic table, depending on what it is exactly.

Sorry that that ended up being quite long, but hope it helps and happy mapping!

Likewise, fixed this too

Hi, there are quite specific ways that collection times need to be written so they can be interpreted correctly by other applications. See here for details:
collection%20times=*
I've fixed this for you.

Hi, you requested a review of this.
First, it would make it easier to review, and help other mappers, if you'd described what detail you'd added in your description, and perhaps briefly why as well.
I think the location and most of the details all seem good. I'm not particularly familiar with railway mapping, but it seems from nearby bits of rail that it would be better tagged with service=crossover, rather than usage=main.

Hi, you requested a review of this and it looks good to me.

Hi, you requested a review of this change. Thanks for adding this path.
I'm not quite sure what you're meaning for the access of this path. You've set the general access to no, but foot to yes.

Hi, you requested a review of this change. Looks good, except that you had the same details on the area and a point, so I've removed the details from the point for you.

Similar comments to the other one, but this looks good too.

Hi, you requested a review of this and it looks good to me.
Some would also add location=roof
generator:source=solar
Also I'm not quite sure about the units. The wiki has kW listed as a suitable units, but I think that with solar kWp (peak) is probably suitable.
generator:output=*

Hi, you requested a review of this edit.
Thanks for adding this, it's important to have even little paths like this mapped to show the connections.
When you added the path, you connected it to the residential area, it's best practice not to do that, so I've disconnected them here for you.

Hi, you requested a review of this, and it looks good to me, although your description does not seem to match what you actually did.

Hi, you requested a review of this and it looks good to me.

Hi, you requested a review of this and it looks good to me.
If it's a cul-de-sac then perhaps it would be better tagged as a residential road, rather than a service road? Depends on its use though.

Hi, you requested a review of this and it looked good, so not quite sure why you've then deleted it?

Hi, you requested a review of this and it looks good to me.