Skip to content
This repository has been archived by the owner on Mar 26, 2021. It is now read-only.

API для регистрации сервисной компании/автосервиса #41

Open
ed-asriyan opened this issue May 20, 2019 · 10 comments
Assignees

Comments

@ed-asriyan
Copy link
Member

ed-asriyan commented May 20, 2019

Actual: #41 (comment)


POST /company/register

{
	"name": "Автосервис у Васи",
	"phone": "8 (499) 763-34-34",
	"address": "Чистопрудный бул., 10, Москва, 101000",
	"login": "vasya68",
	"password": "04031968"
}
@ed-asriyan ed-asriyan assigned YAXo-O and wolf1996 and unassigned YAXo-O May 20, 2019
@wolf1996
Copy link
Contributor

Может разбить на два этапа, как на некоторых сайтах? сначала логин пароль, а потом - заполнение данных акаунта. (Хотя впринципе пофиг, потом можно так сделать)

@ed-asriyan
Copy link
Member Author

ed-asriyan commented May 28, 2019

@wolf1996 в чём преимущество разделения?

@wolf1996
Copy link
Contributor

разделение авторизационной сущности и логической сущности?

@ed-asriyan
Copy link
Member Author

Тогда у нас появится состояние "неактивированного" аккуанта между двумя этими запросами. Оно нам нужно?

@ed-asriyan
Copy link
Member Author

ed-asriyan commented May 31, 2019

Разделение можно "скрыть" внутри хендлера

@wolf1996
Copy link
Contributor

да это то понятно, я про логическую сторону этого вопроса, у нас получится, действительно, что компания как источник данных и компания как потребитель данных это абсолютно разные компании. А это выглядит как-то криво. Ну да пох. Сейчас сделаем в один запрос, потом сделаем в несколько

@ed-asriyan
Copy link
Member Author

ed-asriyan commented May 31, 2019

Я тебя понял.
Давай тогда сделаем регистрацию компании в рамках уже зарегистрированного пользователя?
То есть юзер уже существует, а к нему привязываем новую компанию. Тогда login и password убираются, а к query параметрам добавляется token

И компания будет сразу закреплена за текущем юзером

@wolf1996
Copy link
Contributor

wolf1996 commented Jun 2, 2019

ну впринципе, да к этому всё и шло

@ed-asriyan
Copy link
Member Author

Итого получается:

POST /company/register
Query params:

  • token: access token
{
	"name": "Автосервис у Васи",
	"phone": "8 (499) 763-34-34",
	"address": "Чистопрудный бул., 10, Москва, 101000"
}

@ed-asriyan
Copy link
Member Author

@wolf1996 тогда нужен ещё один хендлер для получения списка всех зареганных компаний на юзера. Забить отдельную ишью на него?

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

No branches or pull requests

3 participants