The conditions technically look fine, however, you’ll want to use schedulers for this as triggers only react to ticket updates / creations. Time events are escalations or pending til events that happen to that moment. That’s not the case if this event is in the past or in the future.
Keep in mind that schedulers require you to ensure that the condition is no longer affected, because other wise the affected ticket will be handled by the same scheduler several times. If you run a scheduler once per day and you’re fine with that, you should be good without additional conditiions.