hexletguides.github.io icon indicating copy to clipboard operation
hexletguides.github.io copied to clipboard

Published 20 hours ago • verified publisher icon Hexlet

add img posts

Open TimurDavlet opened this issue 2 years ago • 9 comments

Добавил миниатюры заглавных изображений на странице со всеми гайдами. Задача: #436

TimurDavlet avatar Sep 08 '22 19:09 TimurDavlet

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

acidmaksim avatar Sep 09 '22 08:09 acidmaksim

А вот ты выше Андрея призвал поревьювить 🙂

ashikov avatar Sep 09 '22 08:09 ashikov

а где скрины или поднятый сайт чтобы посмотреть?)

amshkv avatar Sep 09 '22 10:09 amshkv

1 2

TimurDavlet avatar Sep 09 '22 12:09 TimurDavlet

@amshkv первый скрин берется картинка из image гайда, например 2021-11-09-interpreter.md второй скрин заглушка, если в гайде не указан image

TimurDavlet avatar Sep 09 '22 12:09 TimurDavlet

да, я это понял по коду, мне был интересен визуал :)

amshkv avatar Sep 09 '22 13:09 amshkv

Надо бы заглушку замутить красивую. Попробую Карину попросить, возможно у неё будет время.

ashikov avatar Sep 09 '22 14:09 ashikov

Переговорил. Поставлю тикет на эту задачу и в начале недели надеюсь запилим заглушку. :)

ashikov avatar Sep 09 '22 14:09 ashikov

Тикет стоит. Ждём пока иллюстратор нарисует заглушку.

ashikov avatar Sep 13 '22 05:09 ashikov

@ashikov Вроде же нарисовали, я тикет посмотрел

acidmaksim avatar Oct 02 '22 17:10 acidmaksim

Да. Спасибо, что напомнил! image @TimurDavlet можно добавлять.

ashikov avatar Oct 03 '22 07:10 ashikov

@TimurDavlet тыц

ashikov avatar Oct 10 '22 01:10 ashikov

@TimurDavlet тут осталось буквально только картинку добавить. Давай уже закроем этот PR

ashikov avatar Oct 15 '22 13:10 ashikov

@ashikov 1-2 дня сделаю

TimurDavlet avatar Oct 17 '22 13:10 TimurDavlet

Спасибо! Я уже докинул коммит и смержил

ashikov avatar Oct 18 '22 03:10 ashikov

Сайт деплоится вручную? А то на сайте я не вижу картинок на главной в данный момент.

trolzen avatar Oct 24 '22 23:10 trolzen

ребята переехали на nextjs и немного потеряли именно этот код

amshkv avatar Oct 26 '22 07:10 amshkv

ясно

trolzen avatar Oct 26 '22 18:10 trolzen

Слушайте, я чего-то не понимаю. Этот PR вмерджен. Бот Vercel говорит, что он регулярно деплоит на guides.hexlet.io, например, здесь. И, похоже, это правда, т.к. изменения из своего PR, который смерджен позднее, я вижу. Но вот картинок на главной не вижу. Почему так? @ashikov @acidmaksim @TimurDavlet @dzencot

trolzen avatar Oct 28 '22 10:10 trolzen

потому что это изменение было вмержено в версию сайта на jekyll, а при переносе на nextjs эту штуку не перенесли @dzencot надо вернуть потерянную функциональность

amshkv avatar Oct 29 '22 04:10 amshkv