Agree with pretty much all your points.
Wanted to add though, bit disappointed we didn't hear more about snoke. Also was expecting more hype about what Rey's background would be but nothing was shown other than that her parents were scavengers which I'm not sure I believe.
When I first saw the track at the casino place I thought it was going to be pod racing.
Big plot hole regarding crashing into ships at light speed and why suicide or Droid controlled ships aren't used for that more often.
Disliked that the rebels were just running the whole movie while getting smaller and smaller. Felt boring and slow sometimes
The shot of the ship crashing into the first order fleet at light speed in silence was amazing.
Mary Poppins Leia was shit. Better ways they could have shown how she was in touch with the force too.
Thought Finn sacrificing himself there would have fitted. Was annoyed with the last minute save move.
Feel like Poe is being set up to lead the rebels if/when Leia dies next movie. What with the whole there's more to it than just being a hero lesson.