O que seu colega falou é mais uma recomendação para a organização do projeto, manutenibilidade, escalabilidade e separação de responsabilidades. A gente sabe que é comum tratar o backend (Django, no seu caso) e o frontend (React) como duas aplicações separadas.

A razão para isso é que ambos têm objetivos diferentes. A separação entre eles se torna ainda mais importante quando se trabalha em equipe, onde diferentes pessoas podem estar trabalhando no backend e no frontend. Se os dois estão em repositórios separados, você pode desenvolver, testar, atualizar e versionar cada um deles de forma independente.

Sua abordagem de servir uma aplicação a partir da outra não é errada. Em alguns casos, pode ser exatamente o que você quer fazer, especialmente para projetos menores. Mas, para projetos maiores, separar o frontend do backend pode ser mais flexível e escalável.

Então, resumindo, você não está fazendo algo ruim, mas apenas está fazendo o que funciona para você e para o seu projeto como um todo.