Ok, so basically there’s no use for the imp and voidwalker unless you sac them for their abilities, because the voidwalker no longer gives a shield so it’s extremely useless now and the imp just dies in most forms of combat.
You can’t really sac the succubus, because you can’t afford to channel the ability in most situations.
Before you would sac the succubus for 15% shadow damage which was really good.
If I sac my succubus, I’d rather get lesser invis than seduce, because seduce is used so you can cast in pvp.
I guess the void walker shield is replaced with the superior dark pact so that’s fine, but maybe give the voidwalker the consume shadows ability from before, only it heals you and your party (so it can be used for something!) and then give us that ability when we sac it, this way warlocks can have an aoe zanadalari racial which would be GREAT in a lot of situations, and you wouldn’t have to buff drain life, because now warlocks would get the sustain they desired.
Other than that warlock feels great as affliction atm.
Alternative main demons for demonology would be great too.