Skip to content
This repository was archived by the owner on Jan 9, 2024. It is now read-only.

Добавил модель для сохранения вакансий. #99

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

TyVik
Copy link
Collaborator

@TyVik TyVik commented Sep 7, 2020

Пока драфт, чтобы было что обсуждать. В общем, let's discuss :)

@TyVik TyVik requested a review from langovoi September 7, 2020 18:26
@TyVik TyVik self-assigned this Sep 7, 2020
@TyVik TyVik marked this pull request as draft September 7, 2020 18:26
binakot
binakot previously approved these changes Sep 9, 2020
@langovoi
Copy link
Member

langovoi commented Sep 10, 2020

  1. выделить компанию в отдельную сущность;
  2. добавить координаты для адреса;
  3. добавить описание компании;
  4. добавить название вакансии;
  5. убрать проектный тип вакансий. Это вообще отдельная тема и кажется у нас они не популярны;
  6. добавить валидацию границ ЗП (верхний порог не больше нижнего порога);
  7. добавить тип "стажировка". Их мы разрешаем публиковать без указания ЗП;
  8. добавить для технологий API, которое будет по подстроке подсказывать существующие технологии в нашей базе. Так добьёмся унификации;
  9. добавить получение списка вакансий с пагинацией и фильтрами по всем полям;
  10. добавить получение конкретной вакансии по id;

@langovoi
Copy link
Member

Не хотим ли мы сделать раздел с флагами, например про ДМС?

@langovoi
Copy link
Member

langovoi commented Sep 10, 2020

Не хотим ли мы добавить уровень квалификации? Intern, Junior, Middle, Senior, Lead (enum)

@langovoi
Copy link
Member

langovoi commented Sep 10, 2020

Еще кажется нужно система статусов вакансий, пока можно сделать enum(ACTIVE, ARCHIVED, DECLINED).
DECLINED могут выставлять только админы, чтобы скрыть дичь, но не удалять из базы.
ACTIVE и ARCHIVED может переключать автор или админ.

По умолчанию ставим ACTIVE

@langovoi
Copy link
Member

Вакансии без ЗП разрешаем публиковать, но тогда нужно их опускать в выдаче в API ниже + не будем их публиковать в канал.

@TyVik TyVik marked this pull request as ready for review September 11, 2020 00:32
@TyVik TyVik marked this pull request as draft September 11, 2020 00:34
@TyVik
Copy link
Collaborator Author

TyVik commented Sep 11, 2020

выделить компанию в отдельную сущность

Поддерживать список будут админы? Какие поля нужны помимо названия, описания, адреса, координат и типа (remote/local)?

добавить название вакансии

Добавил

убрать проектный тип вакансий. Это вообще отдельная тема и кажется у нас они не популярны

Я себе ищу людей именно на проектной основе. И ко мне человека 3 обращались именно за разовой помощью, но никто вакансию так и не опубликовал. У меня сложилось впечатление, что просто не умеют описывать.

добавить валидацию границ ЗП (верхний порог не больше нижнего порога)

Добавил

добавить для технологий API, которое будет по подстроке подсказывать существующие технологии в нашей базе.

Добавлю

добавить получение списка вакансий с пагинацией и фильтрами по всем полям

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

добавить получение конкретной вакансии по id

Это есть, но не отражено в тестах

Не хотим ли мы сделать раздел с флагами, например про ДМС?

Пока - нет :) Не ясен набор флагов, и кто их будет поддерживать. Можно завести отдельный тикет на это.

Не хотим ли мы добавить уровень квалификации?

Сложно - могут начаться комбинации. Например, ищут middle+/senior или middle, но готовы и junior взять обучить. Давай это делать уже отдельно в следующем релизе?

Еще кажется нужно система статусов вакансий

Добавил

добавить тип "стажировка". Их мы разрешаем публиковать без указания ЗП
Вакансии без ЗП разрешаем публиковать

Сложно всё сделать с первого раза. Давай отложим на следующий релиз?

@langovoi
Copy link
Member

langovoi commented Oct 3, 2020

Я себе ищу людей именно на проектной основе. И ко мне человека 3 обращались именно за разовой помощью, но никто вакансию так и не опубликовал. У меня сложилось впечатление, что просто не умеют описывать.

Просто мне кажется не стоит мешать в кучу все. Наверно и со стажировками я погорячился, их тоже стоит делать отдельно.

Сложно - могут начаться комбинации. Например, ищут middle+/senior или middle, но готовы и junior взять обучить. Давай это делать уже отдельно в следующем релизе?

Я беру за референс вакансии на Хабр карьере и мне кажется у них вполне удобная форма вакансий. Комбинаций быть не должно, пусть заводят несколько вакансий на разный уровень квалификации. Там и требования разные и компенсация.

Квалификации на Хабр Карьеер: https://career.habr.com/info/divisions_and_qualifications
Кстати у них ещё есть специализация, тоже полезно для классификации и дальнейшего поиска для пользователей.

Скрин формы Хабр карьеры

@langovoi
Copy link
Member

langovoi commented Oct 3, 2020

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

Поиск - отдельная тема, согласен.
А вот фильтр например по навыкам - полезно.

Да, технологии надо переименовать в навыки. Вакансии разные могут быть, где не только разработка, но около неё.

@langovoi
Copy link
Member

langovoi commented Oct 3, 2020

Поддерживать список будут админы?

Пользователь при публикации может либо выбрать компанию (если имеет к ней отношение), либо создать новую (станет ее владельцем).

Отношение пользователя к компании пока делаем просто - если владелец, может ее использовать для вакансии.

Какие поля нужны помимо названия, описания, адреса, координат и типа (remote/local)?

Все перечисленное, кроме типа (пусть пока в вакансиях компании эта информация живёт).

@langovoi
Copy link
Member

langovoi commented Oct 3, 2020

Кстати мы совсем забыли что у вакансии должен быть список каких-то контактов.
Предлагаю сделать возможность указать email, telegram, телефон.

@langovoi
Copy link
Member

langovoi commented Oct 3, 2020

Я завтра все подитожу и нарисую в Miro финальную структуру и пользовательский путь. А то мне кажется мы не много в отрыве от реального использования проектируем вакансии.

@TyVik
Copy link
Collaborator Author

TyVik commented Oct 3, 2020

Кстати мы совсем забыли что у вакансии должен быть список каких-то контактов.
Предлагаю сделать возможность указать email, telegram, телефон.

Можно ограничиться ссылкой на пользователя у нас в базе :) А ему уже в свою очередь добавить все эти поля.

@TyVik
Copy link
Collaborator Author

TyVik commented Oct 3, 2020

А как насчёт кусочек про компании сделать в рамках отдельного тикета? Он необходим для вакансий, но может использоваться и как отдельная база компаний Краснодара. Там же продумать связь с пользователем-администратором, возможности валидации и т.п.

@langovoi
Copy link
Member

langovoi commented Oct 3, 2020

А как насчёт кусочек про компании сделать в рамках отдельного тикета? Он необходим для вакансий, но может использоваться и как отдельная база компаний Краснодара. Там же продумать связь с пользователем-администратором, возможности валидации и т.п.

Ты предлагаешь сначала сделать компании, а потом вернуться и доделать вакансии?

@langovoi
Copy link
Member

langovoi commented Oct 3, 2020

Можно ограничиться ссылкой на пользователя у нас в базе :) А ему уже в свою очередь добавить все эти поля.

Лучше оставить это в вакансии. Точно бывают случаи, когда почта какая-то общая для вакансий или телефон/телеграмм рекрутера из компании, а публикует вакансию кто-то другой.

@TyVik
Copy link
Collaborator Author

TyVik commented Nov 1, 2020

Обновил PR. Насколько я понял, остались нерешённым 2 пункта:

  1. Право пользователей создавать вакансии Добавил модель для сохранения вакансий. #99 (comment). Прелагаю в tg обсудить, т.к. там могут быть заморочки со стороны логики приложения.
  2. Фильтры для поиска. Их я предлагаю вынести в отдельную задачу. Там несложно, но много копипасты. А перед этим хочется быть уверенным, что все поля уже описаны в модели. (Ну и PR и так уже большим вышел)

@TyVik
Copy link
Collaborator Author

TyVik commented Nov 1, 2020

Перед мерджем также мне надо не забыть проадаптировать существующий EnumField, настроить админку и сквошнуть миграции в career.

@langovoi langovoi force-pushed the master branch 2 times, most recently from c7d47dd to a9f775e Compare December 3, 2023 18:44
@langovoi langovoi force-pushed the master branch 4 times, most recently from 9c5d7af to 4ab092f Compare December 3, 2023 18:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants