O que você pode fazer é colocar tudo isso em uma única função. Segue um exemplo:
const firstnameValue = firstname.value()
if(!firstnameValue){
errorValidation(firstname, "Preencha essa campo :(")
} else {
successValidation(firstname);
window.location = "thanks.html"
}
}```
Observação: O ponto de exclamação inverte um valor booleano(true/false), ou seja, caso o firstName esteja vazio colocando o ! antes irá retornar que está vazio, que não há valor, caso colocasse somente if(firstName) estaria dizendo "se firstName existir..." mas colocando if(!firstName) você vai estar dizendo "se firstName não existir...". No else irá usar a função successValidation e em seguida redirecionar para a página de agradecimento
function validateAndSend(){
const firstnameValue = firstname.value()
if(!firstnameValue){
errorValidation(firstname, "Preencha essa campo :(")
} else {
successValidation(firstname);
window.location = "thanks.html"
}
}
Observação: O ponto de exclamação inverte um valor booleano(true/false), ou seja, caso o firstName esteja vazio colocando o ! antes irá retornar que está vazio, que não há valor, caso colocasse somente if(firstName) estaria dizendo "se firstName existir..." mas colocando if(!firstName) você vai estar dizendo "se firstName não existir...". No else irá usar a função successValidation e em seguida redirecionar para a página de agradecimento