The React docs state under componentWillMount():
Avoid introducing any side-effects or subscriptions in this method. For those use cases, use componentDidMount() instead.
And under componentDidMount():
If you need to load data from a remote endpoint, this is a good place to instantiate the network request.
So I'd say it's pretty clear that the advice from the video is not current best practice.