Lemme,
Дробление может быть очень хорошим, правда бывает и избытычным, но так вроде не плохо

...
promise.catch(this.onGetDataError.bind(this));
...

биндить соответстввенно надо если нужен контекст, в противном случае лучше сделать статичный метод.

В 171 аргумент не нужен, я просто забыл его убрать, когда не было навигации, я так проверял переход между страницам.

num = num < 1 || typeof num !== 'number' ? 1 : num >= this.getDataFromStorage().pages.length ? this.getDataFromStorage().pages.length : num;

Почему ничего? Может я в чем то заблуждаюсь, но:

function test(num = 1){
	var numPages = 10;

	if (num < 1 || typeof num !== 'number') {
		num = 1;
	}
	else if (num >= numPages){
		num = numPages;
	}
	return num;
}

console.log(test(5)) // 5
console.log(test(100)) // 10
console.log(test('smth')) // 1

Если рассуждать так, то да, ничего 107 строка не делает, но я лишь решил перестраховаться, вдруг, потом захочется несколько изменить пагинацию. Например слушать изменение хэша.

т.е

<li><a href="#page=10">10</a></li>

В любом случае, спасибо за подсказку =)