search

bem / bem-history

BEM wrap for History API
https://bem.info/libs/bem-history/
Other
23 stars 21 forks source link

Доработки в uri #15

Closed ololobus closed 10 years ago

ololobus commented 10 years ago

Не парсим ";" как разделитель параметров в запросе Uri.decode пытается декодировать криво закодированные символы Разделение параметров на пары key-value: &param=p1;p2=v2 -> { param: ["p1;p2=v2"] } Не добавляем хэшбэнг сразу после инициализации

tadatuta commented 10 years ago

@ololobus готов влить, если PR протестирован.

кстати, давайте в принципе определимся с версионированием bem-history. сейчас есть проблема с тем, что у нас уже есть несколько релизов 1.x и мы фактически не можем версионироваться по semver, т.к. есть еще ветка v2 :(

какие вижу варианты:

  1. версионировать ветку v1 в рамках v1.x.y, где x считать мажорными изменениями, а y инкрементировать в случае минорных или патчевых. при этом для версии на основе bem-core с появлением стабильной версии вести честное версионирование по semver, т.е. будут версии v2, v3 и т.д.
  2. отказаться от развития v1 снаружи и по аналогии с bem-components развивать только версию на основе bem-core по semver.
  3. не плодить лишних библиотек и унести v2 блоки из bem-history в bem-core, а в этом репозитории развивать версию на bem-bl по semver (поставить v2.0.0 на мерж-коммит этого PR).

// cc @veged @dfilatov @mishanga

veged commented 10 years ago

версия не на bem-core нам вроде как всё-равно нужна, версионировать её нужно по semver

про внесение bem-history@v2 в bem-core я совсем не уверен, пока мне кажется лучше завести bem-history2 и там тоже версионировать по semver

ololobus commented 10 years ago

@tadatuta протестирован

tadatuta commented 10 years ago

@veged если есть уверенность, что этот код действительно нужен снаружи и будет развиваться, то есть еще вариант внести его в bem-bl, а bem-history оставить для bem-core. при таком варианте в ветке v1 оставляем все as is, в readme пишем, что код переехал в bem-bl и продолжаем развивать v2 и далее.

заводить bem-history2 не хочется. тем более, что bem-components осталось без чиселки, хотя ситуация была ровно аналогичная.

veged commented 10 years ago

с bem-components всё-таки было не тоже самое — поменялась суть библиотеки

в чём проблема завести чиселку? если по факту так и есть — новая эпоха развития, с сохранением развития предыдущей

tadatuta commented 10 years ago

обсудили с @veged голосом, договорились, что при необходимости развивать библиотеку на основе bem-bl и обновлять мажорные версии, мы можем внести ее внутрь как часть Islands, а снаружи продолжить развивать только bem-core-based версию (начиная с ветки v2 и далее по семверу аналогично тому, как сделано с bem-components).