search

yandex-ui / noscript

Noscript: JavaScript MVC Framework for building SPA
http://yandex-ui.github.io/noscript/
MIT License
34 stars 22 forks source link

fix replace, redirect and back #588

Closed juleari closed 8 years ago

vitkarpov commented 8 years ago

А про что этот пул?

juleari commented 8 years ago

про сломанную историю в случае редиректа

допустим, мы на странице 1, и хотим перейти с неё на страницу 2, с которой есть редирект на страницу 3.

в go вызовется ns.page.redirect который в итоге заменит страницу 1 на страницу 3. помимо этого он ещё и удалит из ns.page._history последний элемент и заменит его на 3

3y3 commented 8 years ago

Тестики бы...

vitkarpov commented 8 years ago

Согласен с тем, то нужен тест на такой кейс.

vitkarpov commented 8 years ago

плюс какая-то беда с jshint, в трависе тесты не проходят. правил в своем пуле https://github.com/yandex-ui/noscript/pull/587/commits/d4cc9196780d0ad378bf05df08cb0d58039c6f9a, видимо, надо в мастер сразу такое отдельным пулом

vitkarpov commented 8 years ago

👍 👍 👍

Katochimoto commented 8 years ago

👍

eljusto commented 8 years ago

👍 👍 👍

vitkarpov commented 8 years ago

@juleari может еще коммиты посквошить, чтобы был один?