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

Документация /vacancy/{vacancy_id}/resumes_by_status #281

Merged
merged 1 commit into from Dec 12, 2017

Conversation

Projects
None yet
3 participants
@ilyakhokhryakov
Member

ilyakhokhryakov commented Dec 5, 2017

No description provided.

@vera-l

несколько вопросов

Show outdated Hide outdated docs/resumes_for_application.md
Show outdated Hide outdated docs/resumes_for_application.md
Show outdated Hide outdated docs/resumes_for_application.md
Show outdated Hide outdated docs/resumes_for_application.md
Show outdated Hide outdated docs/resumes_for_application.md
Show outdated Hide outdated docs/resumes_for_application.md
@ilyakhokhryakov

This comment has been minimized.

Show comment
Hide comment
@ilyakhokhryakov

ilyakhokhryakov Dec 5, 2017

Member

Когда этот флаг предполагается использовать?

Если выдается флаг is_incomplete=true, резюме нужно дозаполнить перед откликом на переданную вакансию. Фактически таким резюме можно откликнуться на такую вакансию как есть, но этому была своя причина и это не приветствуется.

Member

ilyakhokhryakov commented Dec 5, 2017

Когда этот флаг предполагается использовать?

Если выдается флаг is_incomplete=true, резюме нужно дозаполнить перед откликом на переданную вакансию. Фактически таким резюме можно откликнуться на такую вакансию как есть, но этому была своя причина и это не приветствуется.

@ilyakhokhryakov

This comment has been minimized.

Show comment
Hide comment
@ilyakhokhryakov

ilyakhokhryakov Dec 5, 2017

Member

Возможно, флаг is_incomplete стоит переименовать в что-нибудь вроде requires_completion

Member

ilyakhokhryakov commented Dec 5, 2017

Возможно, флаг is_incomplete стоит переименовать в что-нибудь вроде requires_completion

@vera-l

This comment has been minimized.

Show comment
Hide comment
@vera-l

vera-l Dec 6, 2017

Contributor

да, про requires_completion - хорошая идея

Contributor

vera-l commented Dec 6, 2017

да, про requires_completion - хорошая идея

@vera-l

This comment has been minimized.

Show comment
Hide comment
@vera-l
Contributor

vera-l commented Dec 6, 2017

@vera-l

This comment has been minimized.

Show comment
Hide comment
@vera-l

vera-l Dec 6, 2017

Contributor

💡 для себя заметка:

  • возможно, переименовать "Резюме, сгруппированные по возможности откликнуться на данную вакансию"
  • сделать англ. версию
  • добавить упоминание сюда
    вакансию, можно воспользоваться [списком подходящих резюме](suitable_resumes.md).
  • кое-где выдаем поле "siutable_resumes_url"- хорошо, что доку не удалили
Contributor

vera-l commented Dec 6, 2017

💡 для себя заметка:

  • возможно, переименовать "Резюме, сгруппированные по возможности откликнуться на данную вакансию"
  • сделать англ. версию
  • добавить упоминание сюда
    вакансию, можно воспользоваться [списком подходящих резюме](suitable_resumes.md).
  • кое-где выдаем поле "siutable_resumes_url"- хорошо, что доку не удалили
@vera-l

vera-l approved these changes Dec 6, 2017

@ilyakhokhryakov

This comment has been minimized.

Show comment
Hide comment
@ilyakhokhryakov

ilyakhokhryakov Dec 6, 2017

Member

засквошил коммиты

Member

ilyakhokhryakov commented Dec 6, 2017

засквошил коммиты

@iriss22

iriss22 approved these changes Dec 6, 2017

Show outdated Hide outdated docs/resumes_for_response.md
@iriss22

случайно зааппрувила, осталось несколько вопросов

@vera-l

vera-l approved these changes Dec 6, 2017

@iriss22

iriss22 approved these changes Dec 6, 2017

@ilyakhokhryakov

This comment has been minimized.

Show comment
Hide comment
@ilyakhokhryakov

ilyakhokhryakov Dec 6, 2017

Member

засквошил еще раз

Member

ilyakhokhryakov commented Dec 6, 2017

засквошил еще раз

@ilyakhokhryakov ilyakhokhryakov changed the title from Черновик документации /vacancy/{vacancy_id}/resumes_for_application to Документация /vacancy/{vacancy_id}/resumes_by_statuses Dec 6, 2017

@vera-l vera-l changed the title from Документация /vacancy/{vacancy_id}/resumes_by_statuses to Документация /vacancy/{vacancy_id}/resumes_by_status Dec 7, 2017

@vera-l

появилось новое

@vera-l

This comment has been minimized.

Show comment
Hide comment
@vera-l

vera-l Dec 7, 2017

Contributor

решили делать без пагинации, так как по смыслу она тут не нужна.

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

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

Contributor

vera-l commented Dec 7, 2017

решили делать без пагинации, так как по смыслу она тут не нужна.

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

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

Show outdated Hide outdated README.md
@vera-l

vera-l approved these changes Dec 7, 2017

Поле | Тип | Описание
---- | --- | --------
requires_completion | boolean | Требуется ли дозаполнить [обязательные поля](resumes.md#author-progress) резюме для отклика на вакансию. Принимает значение `true` только в случае, если в вакансии не установлен флаг «принимать неполные резюме» и резюме является неполным; в противном случае — `false`.

This comment has been minimized.

@vera-l

vera-l Dec 7, 2017

Contributor

обязательные ли тут поля из https://github.com/hhru/api/blob/master/docs/resumes.md#author-progress или рекомендованные?

@vera-l

vera-l Dec 7, 2017

Contributor

обязательные ли тут поля из https://github.com/hhru/api/blob/master/docs/resumes.md#author-progress или рекомендованные?

This comment has been minimized.

@ilyakhokhryakov

ilyakhokhryakov Dec 7, 2017

Member

обязательные, mandatory

@ilyakhokhryakov

ilyakhokhryakov Dec 7, 2017

Member

обязательные, mandatory

Описание полей смотрите в [выдаче полного резюме](resumes.md#resume-fields).
Дополнительно для каждого резюме выдается поле `requires_completion`, зависящее от наличия флага «принимать неполные резюме» в требуемой вакансии.

This comment has been minimized.

@iriss22

iriss22 Dec 7, 2017

Contributor

кажется либо в этом предложении не нужно писать "зависящее от наличия флага «принимать неполные резюме» в требуемой вакансии" Либо не нужна таблица ниже, а просто описать это в предложении

@iriss22

iriss22 Dec 7, 2017

Contributor

кажется либо в этом предложении не нужно писать "зависящее от наличия флага «принимать неполные резюме» в требуемой вакансии" Либо не нужна таблица ниже, а просто описать это в предложении

This comment has been minimized.

@vera-l

vera-l Dec 7, 2017

Contributor

да, можно просто

Дополнительно для каждого резюме выдается поле requires_completion:
и тут таблица

@vera-l

vera-l Dec 7, 2017

Contributor

да, можно просто

Дополнительно для каждого резюме выдается поле requires_completion:
и тут таблица

This comment has been minimized.

@iriss22

iriss22 Dec 7, 2017

Contributor

Хотя, наверное, лучше оставить так

@iriss22

iriss22 Dec 7, 2017

Contributor

Хотя, наверное, лучше оставить так

@@ -151,7 +153,8 @@ GET /vacancies/{vacancy_id}/suitable_resumes
"rtf": {
"url": "https://hh.ru/api_resume_converter/14831542000d1f366b4c5a6a751b329b70039e/ИвановИванИванович.rtf?type=rtf"
}
}
},
"requires_completion": false

This comment has been minimized.

@iriss22

iriss22 Dec 7, 2017

Contributor

в этом списке нет резюме с "requires_completion": true?

@iriss22

iriss22 Dec 7, 2017

Contributor

в этом списке нет резюме с "requires_completion": true?

This comment has been minimized.

@ilyakhokhryakov

ilyakhokhryakov Dec 7, 2017

Member

в теории может быть, в примере в доке нет

@ilyakhokhryakov

ilyakhokhryakov Dec 7, 2017

Member

в теории может быть, в примере в доке нет

This comment has been minimized.

@iriss22

iriss22 Dec 7, 2017

Contributor

а раньше такие резюме ведь сюда не попадали?

@iriss22

iriss22 Dec 7, 2017

Contributor

а раньше такие резюме ведь сюда не попадали?

This comment has been minimized.

@ilyakhokhryakov

ilyakhokhryakov Dec 7, 2017

Member

попадали

@ilyakhokhryakov

ilyakhokhryakov Dec 7, 2017

Member

попадали

This comment has been minimized.

@ilyakhokhryakov

ilyakhokhryakov Dec 7, 2017

Member

откликаться-то ими можно

@ilyakhokhryakov

ilyakhokhryakov Dec 7, 2017

Member

откликаться-то ими можно

This comment has been minimized.

@iriss22

iriss22 Dec 7, 2017

Contributor

ну тогда ок

@iriss22

iriss22 Dec 7, 2017

Contributor

ну тогда ок

@iriss22 iriss22 merged commit 03ba348 into master Dec 12, 2017

@vera-l vera-l deleted the resumes-for-application branch Dec 20, 2017

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