That sums it up well. The entire war in-universe is just a setup for the Shadowlands story, but they needed to sell you on the expansion that actually existed on the time, so they went with the angle of a genocidal struggle for existence that gave us highlights like a focus on night elves burning alive and mothers being speared to walls in front of their children. Hell, this is the toned down version from beta/PTR builds that had death camps set up for night elf civilians and Alliance vulpera massacre squads. And this portrayal of the most brutal war in the series doesn't have any conclusion: Saurfang gets a conclusion to his story, but Saurfang is not the main character and this is not a story solely about individuals.
The implied attitude seemed to be that we can brush off the war because it was just a setup for Shadowlands, but that doesn't work if you're trying to take the story seriously as something that actually happened, that characters should react to and not just exist as a series of events that vanished from history as soon as Shadowlands released. It failed as a plot in and of itself and it failed on a thematic level. For as much deserved focus that Shadowlands gets for the damage it did to the overall setting and cosmology of Warcraft, BfA is its ugly older brother that casts a looming shadow over the ongoing plot as an unresolved moral nightmare that the writers are still running the hell away from.