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

add filter by skill first_name last_name email resume_name #413

Conversation

usernaimandrey
Copy link
Contributor

Предлагаю свое решение данной проблемы:

Можно делать поиск по имени или фамилии или email поиск по вхождению
Снимок экрана от 2022-12-13 12-17-25

Можно сделать поиск по Названию резюме - поиск по вхождению

Снимок экрана от 2022-12-13 12-17-06

Можно по направлению:

Единственное надо либо как то промигрировать резюме в плане тегов или как то анонсировать чтобы пользователи отредактировали резюме с учетом тегирования

Снимок экрана от 2022-12-13 12-17-35

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

Снимок экрана от 2022-12-13 12-17-51

@usernaimandrey
Copy link
Contributor Author

Как вариант можно добавить tomselect чтобы был поиск в селекте

@emp7yhead
Copy link

@amshkv можешь посмотреть пожалуйста?

@fey
Copy link
Collaborator

fey commented Dec 28, 2022

А напомни, какую проблему решаем? Если поиск по стеку, то поиск по фио выглядит избыточным.

@usernaimandrey
Copy link
Contributor Author

А напомни, какую проблему решаем? Если поиск по стеку, то поиск по фио выглядит избыточным.

#213
#283
вот эти ишю
на мой взгляд как альтернативный вариант, там же еще можно и по почте пользователя искать, тем более по началу поиск по тэгам будет мало эффективен, либо надо как то грамотно промигрировать резюме, что бы они уже содержали тэги, тогда возникает вопрос откуда брать информацию для тэга, из названия или из навыков, и как это разпарсить
Возможно стоит спросить у тех кто этим будет пользоваться, у Екатерины и Алисы, возможно им такой поиск будет удобнее.

Copy link
Contributor

@grozwalker grozwalker left a comment

Choose a reason for hiding this comment

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

В целом по тех части все супер! Надо немножко подправить имя

# frozen_string_literal: true

module TagResumePresenter
def tags_sorted_list(sort_method = 'ASC')
Copy link
Contributor

Choose a reason for hiding this comment

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

единственное минизамечание, что название этого метода не очень соответствует тому, что здесь выбирается. Здесь же мы берем все тэги направления?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

единственное минизамечание, что название этого метода не очень соответствует тому, что здесь выбирается. Здесь же мы берем все тэги направления?

Да берем все тэги направления относящиеся к резюме, да можно наверное переименовать all_tags или all_tags_sorted_list

Copy link
Contributor Author

Choose a reason for hiding this comment

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

В целом по тех части все супер! Надо немножко подправить имя

@grozwalker Спасибо

Copy link
Contributor

Choose a reason for hiding this comment

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

не, не по идее надо назвать как-то: directions_tags, чтобы в названии было именно directions, чтобы было понятнее.
Или я что-то недопонимаю?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

да все верно, у нас здесь теги направления, а есть еще тэги skills, directions_tags - более говорящее имя

Copy link
Contributor Author

Choose a reason for hiding this comment

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

https://github.com/Hexlet/hexlet-cv/blob/main/app/presenters/tag_presenter.rb#L4
здесь бы тогда наверное тоже надо переименовать, так как есть тэги directions и technologes

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Андрей @grozwalker я поправил.

@usernaimandrey usernaimandrey force-pushed the add-skills-by-which-you-can-filter-resumes branch from d6ed994 to 05fd38b Compare December 29, 2022 15:24
Copy link
Contributor

@grozwalker grozwalker left a comment

Choose a reason for hiding this comment

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

Вроде кроме названий у меня больше небыло вопросов, так что давай засквошим в один коммит и я смерджу

@grozwalker grozwalker merged commit 44d9ebc into Hexlet:main Jan 10, 2023
@grozwalker
Copy link
Contributor

Спасибо!)

@usernaimandrey
Copy link
Contributor Author

Спасибо!)

Всегда рад помочь)

amd-9 pushed a commit to amd-9/hexlet-cv that referenced this pull request Jan 17, 2023
* add filter by skill first_name last_name email resume_name
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants