Bug: Отсутствуют ссылки на pull requests

[ajib6ept]

Ожидаемое поведение

Активные ссылки на pull requests

Текущее поведение

Отсутвуют активные ссылки на pull requests

Возможные решения

Дейсвия по воспроизведению

1. На главной странице перейти в раздел "Пул-реквесты"
2. Нажать на "Статус" и сдеать сортировку по убыванию
3. В столбце "Название" отсутствует активная ссылка (также поле "Статус" пустое)

Информация о вашей системе

Информация об установленных зависимостях

pip freeze

[fey]

@zluuba попробуйте взять задачку.

[zluuba]

Спасибо, беру в работу

[zluuba]

Доброго дня,

Как я понимаю, этот баг вызван тем, что в самой базе данных отсутствуют статусы и названия этих pull requests (ссылки на них, кстати, подгружаются как обычно, но не отображаются из-за отсутствия названия), так как текущая реализация позволяет создать contribution с типом “pull request” без "IssueInfo" (по крайней мере локально в админке), то есть у объекта “pr” (pull request) может отсутствовать свойство “info”, из которого и берутся название и статус.

Так как pull requests с пустыми полями относятся к репозиториям, которых нет в разделе “Организации” (их список внизу), плюс к тому они архивированы уже длительное время, я бы предложила три варианта решения:

1. Не отображать их на странице в принципе при помощи обычного if statement в самом шаблоне “pull_requests_list.html”;
2. Реализовать default values при отсутствии названия и статуса, например “Pull request #42”, где на месте числа подставлялся бы id;
3. Удалить эти пул реквесты из бд (или изменить, но в любом случае это была бы работа с самой бд).

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

Дайте, пожалуйста, обратную связь по информации выше (:

Репозитории с забагованными pull requests:
https://github.com/Hexlet/osx-project-1/
https://github.com/Hexlet/osx-project-2/
https://github.com/Hexlet/osx-project-3/
https://github.com/Hexlet/osx-week7-project/

[fey]

@zluuba Оо, классно проделанная работа!) Можете подсказать, на какие ПРы посмотреть, чтобы понять, что они забагованы?
Я думаю, что можно использовать 2 вариант с подстановкйо дефолтного названия ПРа.

[zluuba]

@fey
вот немного забагованных pull requests: один, два, три.
Здесь первые три из списка на сайте.
А вообще все pull requests из репозиториев, ссылки на которые я оставила выше, забагованы.

Тогда завтра реализую второй вариант, спасибо за высокую оценку!)

[fey]

Понял, может это просто очень старые пры и в этом дело. Удалять их не вижу смысла =)