Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Get all user registration activity #174

Closed
zhabinka opened this issue Jul 25, 2022 · 7 comments · Fixed by #191
Closed

Get all user registration activity #174

zhabinka opened this issue Jul 25, 2022 · 7 comments · Fixed by #191
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@zhabinka
Copy link
Contributor

No description provided.

@zhabinka zhabinka changed the title Вытянуть всю активность пользователей при регистрации Получить всю активность пользователей при регистрации Jul 25, 2022
@acidmaksim
Copy link
Contributor

acidmaksim commented Jul 26, 2022

You need all activity from GitHub to be pulled to the user's page when registering or linking the account to hexlet-friends.

Activity:

  • Pull request;
  • Commits;
  • Issues;
  • etc.

@neihaoo
Copy link
Contributor

neihaoo commented Aug 29, 2022

Здравствуйте, хочу попробовать взять данную задачу в работу.

@zhabinka
Copy link
Contributor Author

Отлично!

@neihaoo
Copy link
Contributor

neihaoo commented Sep 5, 2022

@zhabinka возник вопрос по задаче. Вопрос связан с тем что у репозитория владелец может быть как пользователь так и организация, сейчас я просто добавил поле owner для модели Repository и связываю его с пользователем если тип владельца пользователь, а не организация. А вообще я думаю что можно воспользоваться Generic Relations, чтобы связать владельца репозитория либо с пользователем либо с организацией. Но тогда там надо рефакторить код в других местах где он завязан на текущей реализации и впринципе это можно оформить и как отдельную задачу, либо сделать в рамках этой отдельным коммитом просто. Как лучше поступуть?

@neihaoo
Copy link
Contributor

neihaoo commented Sep 9, 2022

Пошел по простому пути пока.

@acidmaksim acidmaksim added enhancement New feature or request good first issue Good for newcomers labels Sep 12, 2022
@acidmaksim acidmaksim changed the title Получить всю активность пользователей при регистрации Get all user registration activity Sep 13, 2022
@acidmaksim
Copy link
Contributor

@neihaoo как дела?

@neihaoo
Copy link
Contributor

neihaoo commented Sep 13, 2022

@acidmaksim нормально, PR отправил, жду проверки.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants