Выводит дату статей красиво #60

pepelsbey commented 5 years ago

А нельзя это как-нибудь с помощью Intl API сделать, попроще?

pepelsbey commented 5 years ago

Тревис падает с ошибкой

error [new/src/templates/articleList.jsx] The page component must export a React component for it to be valid

pepelsbey commented 5 years ago

Повесил лейбл «в работе», чтобы было понятно, что игра в полном разгаре и мёржить не надо :)

pepelsbey commented 5 years ago

Вот вам и легкотня 😥

h4 commented 5 years ago

Я бы не городил велосипед на регекспах, а взял бы локализацию из date-fns или Luxon.

pepelsbey commented 5 years ago

@h4 а тут Intl API не поможет? На острие ножа, так сказать.

pepelsbey commented 5 years ago

Нет, серьёзно, а может и правда toLocaleString здесь будет достаточно? Отдать конечно нужно будет в правильном формате, но вывести будет проще.

h4 commented 5 years ago

@pepelsbey вполне

const d = new Date();
d.toLocaleDateString("ru-RU", {day: "numeric", month: "long", year: "numeric"});

// "17 января 2019 г."

pepelsbey commented 5 years ago

@ishpartko а давай переделаем на toLocaleDateString, будет совсем красиво :)

h4 commented 5 years ago

@ishpartko я в #65 пофиксил вещи, из-за которых тревис падал.

pepelsbey commented 5 years ago

@h4 пулреквест для пулреквеста! Остановись, что ты делаешь.

h4 commented 5 years ago

Ну надо же как-то было разобраться, в чём там беда со сборкой :)

On Thu, 17 Jan 2019 at 19:17, Vadim Makeev notifications@github.com wrote:

@h4 https://github.com/h4 пулреквест для пулреквеста! Остановись, что ты делаешь.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/web-standards-ru/new/pull/64#issuecomment-455232419, or mute the thread https://github.com/notifications/unsubscribe-auth/AAaPxwcK9aBBE7IrWiVxTLiJSbR1XQjXks5vEKIwgaJpZM4aFN6_ .

ishpagin commented 5 years ago

Ну надо же как-то было разобраться, в чём там беда со сборкой :) … On Thu, 17 Jan 2019 at 19:17, Vadim Makeev @.***> wrote: @h4 https://github.com/h4 пулреквест для пулреквеста! Остановись, что ты делаешь. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#64 (comment)>, or mute the thread https://github.com/notifications/unsubscribe-auth/AAaPxwcK9aBBE7IrWiVxTLiJSbR1XQjXks5vEKIwgaJpZM4aFN6_ .

Сейчас еще раз форкнул и заново попробую на чистой версии, проверка падала из-за переносов строк(у операционок разный разделительный символ в конце), конфиг настроен на мак, и по идее моя IDE должна кодхватывать конфиги, но почему-то она отправляла сюда CRLF.

Сейчас глянем что "с нуля" будет

pepelsbey commented 5 years ago

Тревис всё ещё красный:

The page component must export a React component for it to be valid

ishpagin commented 5 years ago

Парни, я не знаю что там было наверняка, но пересоздание форка помогло.

Проверьте пожалуйста #67

Этот удалите

pepelsbey commented 5 years ago

Чёртова магия 😮

web-standards-ru / new

Выводит дату статей красиво #60 #64