search

bem-archive / bem-site-engine

BEM CMS based on Express framework.
25 stars 8 forks source link

Implemented middleware__response-headers for setting common response headers #196

Closed SwinX closed 9 years ago

SwinX commented 9 years ago

@tormozz48 посмотри пожалуйста

tavriaforever commented 9 years ago

@SwinX @gela-d Нужно хорошо потестить сайт. Так как эти заголовки устанавливаются на все реквесты которые могут происходить на сайте, и форум с аяксом и примеры блоков и html код примера.

SwinX commented 9 years ago

@tavriaforever Как предлагается потестить? Ходить руками по ссылкам кажется не самой продуктивной идеей =)

tavriaforever commented 9 years ago

@SwinX Есть идеи лучше?)

tormozz48 commented 9 years ago

@SwinX По коду пулл-реквеста все :ok:

По проверке: можно пооткрывать страницы разного типа:

  1. Обычные статьи
  2. Блоки (вкладки с примерами тоже)
  3. Страницы форума
SwinX commented 9 years ago

Покликал по сайту в случайных местах - вроде бы всё работает точно также, как на ru.bem.info. Но я всё равно как-то не уверен в такой проверке.

SwinX commented 9 years ago

@tormozz48 что с этим PR ? Можно мержить?

tormozz48 commented 9 years ago

@SwinX да