I don't have a problem with the stories of Leah and Geordi getting together or Riker and Worf getting on each others nerves, except that Deanna had passed away.
It looked like Data did have emotions when he said that his assistant made him laugh.
I don't have a problem with it per say. It just has the flavor of being a rush job. Suddenly Riker & Worf have beef over Deana, when they specifically addressed that possible conflict prior, unless something new happened... which we never get any history on
& there's nothing wrong with Geordi & Leah getting together per say, but it just smacks of them scrambling into the character's history, last minute, to try to drum up some kind of future for him, like "let's give him a family with some girl he knows... Oh wait, he's a dork who never had a girlfriend... Eh well, we'll figure something out"
I think the story with Q was great, it connected the entire series from first to last episode and gave TNG "trial of humankind" storyline through the series.
It was a solid attempt, but the actual trial was that we're a savage race, & I think it is kind of fitting that it becomes more about whether we're capable of broadening our horizons existentially, but time travel is his big test? He literally dumped him back in time once already, & gave him a lesson about tampering very recently. Plus the fact that Picard has already had more time travel & time traveler dealings than nearly anyone. It's a bit of a soft ball for him, as tests go, IMHO. Frankly, I'd imagine that the only reason he struggles with it at all is because he's suffering from a disorder that has weakened his mind some.
That said... The real reason the test was that setup specifically, wasn't so much as a reflection on the humanity trial, but as a way to physically feature the whole of the show in the episode, past, present, future, as a celebration of it, & in that sense, I do love it, because it's fun to see them trying to play back to day 1.