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

Создать миграцию модели пользователей #6

Closed
artesby opened this issue Feb 23, 2016 · 6 comments
Closed

Comments

@artesby
Copy link
Contributor

artesby commented Feb 23, 2016

#5

в модель User добавить поля:

  • registration_date (date)
  • last_visit (datetime)

связать отношением 1:1 с моделями:

1:m

m:m

Теги: model

@malinink
Copy link
Owner

@artesby
Предлагаю в начале каждой задачи делать ссылку на родительскую задачу #4 в нашем случае
Следует упомянуть что модель users уже есть, а то, что тут указываете Вы - непосредственно её дополнение, т.е. только миграция.
Теги как я понял проставлять он не дает? (тогда давайте писать в конце предполагаемые теги)

По содержимому на текущий момент:

  • модель User уже есть, предлагаю отталкиваться от того что в ней находится, т.е. переименовать модели по которым выставлены задачи.

Предлагаю поправить только эту задачу и поглядеть что выйдет, а потом уже взяться за остальные.

@artesby
Copy link
Contributor Author

artesby commented Feb 24, 2016

Я же и написал создать миграцию модели, в других случаях я писал создать и модель и миграцию. просто написал, что по итогу пользователь должен содержать, вместо перечисления что удалить\переименовать.
Лейбл поставить не дает, да.

модель User уже есть, предлагаю отталкиваться от того что в ней находится, т.е. переименовать модели по которым выставлены задачи.

это вообще не понял)

@malinink
Copy link
Owner

@artesby Суть в том, что в текущей модели User есть уже определенный набор полей, как то

  • email
    У Вас же это поле находится в модели PersonalInfo
    Я пытался высказать мысль, что это поле перенести так вот просто не получится, оно должно остаться в модели User.
    В связи с этим определенно надо будет внести коррективы в модели и их поля.

@malinink malinink added the model label Feb 25, 2016
@artesby
Copy link
Contributor Author

artesby commented Feb 25, 2016

но это для нас тоже кучу вопросов порождает, почему сложно перенести, что еще в схемке нужно менять (пароль?), почему мы это не увидели на паре

@malinink
Copy link
Owner

Потому что не подумали об этом, погодите:

  • Надо исходить из того что у нас уже есть, т.к. это базовый функционал Ларавеля, нет большого смысла придумать свои "уникальные" модели, а потмом переписывать функционал аутентификаци.
  • Т.е. модель User уже есть

Моё предложение User объект использовать как средство для идентификации пользователя. И опять давайте я не буду думать наперед, будете налетать на проблемы сами. тогда #7 пока что лишняя. Не используем и убираем из всех связей.

@artesby
Copy link
Contributor Author

artesby commented Mar 3, 2016

возьмусь за это задание

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

No branches or pull requests

2 participants