I thought it was bad for the reasons others have pointed out. A few things to add that were probably said somewhere:
1) I feel that at least a few more minor characters should have died. It was a major gun battle, and only Sasha died...and pre-battle to boot. They could spare Dwight and Negan, that's fine, but the garbage pals leader (or her second in command, the old guy) or Simon were good targets for death. The battle just has no impact when only people you've never seen before (or maybe once or twice briefly) are the ones dying in it.
2. The worst part of the final battle was how Rick and crew appeared to be winning or at least holding their ground after the coffin debacle and zombie Sasha. Then the next thing you know, they are easily defeated and Rick and Carl are on their knees. Of course, don't wait to actually defeat everyone before giving a speech, Negan, as we still hear gunshots in the distance, just ham it up with the battle still apparently happening, but still victory was ensured somehow? And I think the problem with the whole battle is how rushed it was. You had so much of the episode focused on flashbacks, and of course setup (when the whole damn season was setup) that you had a rushed final battle that made no sense, from how quickly one side gained an advantage, to the silly and easy escape made possible with just a few homemade smoke bombs. Well hey, at least the season didn't end on a ridiculous cliffhanger, so there's that. And maybe things will actually happen next season unlike this one. They better, or ratings will continue to fall.