Skip to content
This repository was archived by the owner on Jul 23, 2025. It is now read-only.

[#360] Show that repos is Forked if Repositories#367

Draft
RubanovDmitriy wants to merge 3 commits into
Hexlet:mainfrom
RubanovDmitriy:issue_360_show_forked_repos
Draft

[#360] Show that repos is Forked if Repositories#367
RubanovDmitriy wants to merge 3 commits into
Hexlet:mainfrom
RubanovDmitriy:issue_360_show_forked_repos

Conversation

@RubanovDmitriy
Copy link
Copy Markdown
Contributor

* Added filter for oranization.name in repositories_list.html
<a href="{{ repository.get_absolute_url }}">
{{ repository.name }}
</a>
{% elif repository.organization.name != 'Hexlet' %}
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

вот эта штука не должна решаться на фронте. Ведь есть репозитории, которые форкнули в Хекслете, а есть проекты-оригиналы для тех. Посмотрите, что может отдавать гитхаб. Врое бы там есть атрибут, который говорит .что репозиторий - форк.

* Added bool field "fork" to Repository model
* Added saving this field to update_or_create_record
* Added check for fork in repositories_list template
* Added bool field "fork" to Repository model
* Added saving this field to update_or_create_record
* Added check for fork in repositories_list template
@fey
Copy link
Copy Markdown
Collaborator

fey commented Nov 21, 2023

Можете подключить еще парочку репозиториев? Можно например

@fey
Copy link
Copy Markdown
Collaborator

fey commented Nov 21, 2023

operations = [
migrations.AddField(
model_name="repository",
name="fork",
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

тк это булево значение, давайте его так и назовем is_fork и дальше ниже по коду.

@fey fey marked this pull request as draft November 27, 2023 12:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants