Skip to content

Вносит правки на странице участника#1324

Open
solarrust wants to merge 3 commits into
mainfrom
fix/cut-https
Open

Вносит правки на странице участника#1324
solarrust wants to merge 3 commits into
mainfrom
fix/cut-https

Conversation

@solarrust

@solarrust solarrust commented Feb 21, 2025

Copy link
Copy Markdown
Member
  1. Выводит обрезанную ссылку, без http(s)://, www. и слэша в конце
  2. Добавляет в ссылки нулевой пробел, чтобы переносилось по знакам пунктуации, а не по одной букве
  3. Поменяла иконку твитора на Х (😭)
  4. Поменяла подчёркивание ссылок с градиента (why?) на text-underline (@skorobaeus нужен твой аппрув)
  5. Заменила заморские «issues» и «pull requests» на знакомые каждому русскоговорящему «ишьюс» и «пулреквесты» 🤓

Локально у меня почему-то в какой-то момент перестали подтягиваться данные по количеству ПРов и ишьюс. Надеюсь, я ничего не сломала 😅

@solarrust solarrust added the улучшение Доработка существующего label Feb 21, 2025
@solarrust solarrust requested a review from skorobaeus February 21, 2025 22:27
@skorobaeus

skorobaeus commented Feb 26, 2025

Copy link
Copy Markdown
Member
  1. Поменяла подчёркивание ссылок с градиента (why?) на text-underline (@skorobaeus нужен твой аппрув)

Однозначно лайк. Оно и работает лучше, чем градиент.

  1. Заменила заморские «issues» и «pull requests» на знакомые каждому русскоговорящему «ишьюс» и «пулреквесты» 🤓

А это, боюсь, не лайк, т.к. нужны склонения: 5 пулреквестов, 4 пулреквеста, 1 пулреквест.

@solarrust

Copy link
Copy Markdown
Member Author

А это, боюсь, не лайк, т.к. нужны склонения: 5 пулреквестов, 4 пулреквеста, 1 пулреквест.

На сколько я вижу, у нас есть инструмент для склонения. Осталось добавить пару форм.

@github-actions

Copy link
Copy Markdown

Превью контента из b9ab283 опубликовано.

@solarrust

Copy link
Copy Markdown
Member Author

Обновила фильтр плюрализации, добавила в него форму few чтобы правильно показывать множественные формы русских слов. Перепроверьте за мной, пожалуйста! @skorobaeus

CleanShot 2025-03-11 at 18 17 51
CleanShot 2025-03-11 at 18 18 02
CleanShot 2025-03-11 at 18 18 14
CleanShot 2025-03-11 at 18 18 23
CleanShot 2025-03-11 at 18 18 49

@solarrust

solarrust commented Mar 11, 2025

Copy link
Copy Markdown
Member Author

Для потомков

Как сделать так, чтобы локально были видны данные по ишьюс/пулреквестам:

  1. Скачиваем файл с кэшем из отдельного репозитория → https://github.com/doka-guide/cache/blob/main/issues.json
  2. В корне папки платформы локально создаём папку cache/, в неё кладём скачанный файлик issues.json.
  3. Комментируем/удаляем условие про прод тут
    if (isProdEnv) {
  4. Profit!

Только не коммитите это, пожалуйста 🙏

@solarrust solarrust self-assigned this Mar 11, 2025

@skorobaeus skorobaeus left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Я проделала всё по твоей инструкции, но ПР и ишью у меня локально не отображаются :( В превью — старая версия. На твоих скриншотах пуллреквесты склоняются правильно.

А вот у слова "ишью" нет ни множественного числа, ни склонения: одно ишью, два ишью, десять ишью, сто пятьдесят девять ишью. Аналогичное слово: интервью. (Заимствования — скользская тропка).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

улучшение Доработка существующего

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants