Напоминания: убрать дубль ManagerEmail, ссылка из настроек; None первым во всех enum
Напоминания: - ProbationEndingReminderDto: ManagerEmail убран — это был унаследованный ReceiverEmail под вторым именем. - Ссылка для ProbationEnding больше не ведёт на саму заявку на приём (она уже закрыта, решать по ней нечего) — берётся из новой настройки RemindersOptions.ProbationReminderUrl и ведёт в раздел, где заводят новую заявку. Подстановок в ней нет: адрес конкретной формы в портал не передать, rptype без rpid/rpsource игнорируется (см. parseRoute в request-portal). Заодно ушли BuildProbationUrl и ненужные больше параметры type/listId. Стайлгайд (CLAUDE.md) — у enum первое значение None: - ReminderFeed: None=0 трактуется как ApprovalOverdue, чтобы вызов без явного выбора работал как раньше. - EmploymentStatus: заодно инвертирован switch в ReminderService — раньше «продолжить» было веткой по умолчанию, и новый None молча означал бы «сотрудник работает». Теперь по умолчанию пропуск с предупреждением. - ConditionsLogic: None — НЕвалидное значение, отвергается валидаторами шага (в том числе новым PatchTemplateRouteStepRequestValidator, его не было вовсе). Поэтому у ConditionsLogic в Create/Update-запросах убрано значение по умолчанию: System.Text.Json в .NET 10 уважает дефолты параметров конструктора, и пропущенное поле молча означало «И». Теперь пропущенное поле даёт None и упирается в валидатор. ЛОМАЮЩЕЕ ИЗМЕНЕНИЕ КОНТРАКТА: клиент, не присылающий conditionsLogic в POST/PUT шага, получит 400 вместо прежнего молчаливого «И». Админка не затронута — она это поле всегда отправляет.
This commit is contained in:
gorbenko.pavel
@@ -4,14 +4,21 @@ namespace Tnsu.ApprovalRoute.Application.Reminders;
|
||||
|
||||
internal enum EmploymentStatus
|
||||
{
|
||||
/// <summary>
|
||||
/// Статус не определяли. EmploymentOf такого не возвращает — значение существует
|
||||
/// только как значение по умолчанию (default(EmploymentStatus)); трактуется
|
||||
/// наравне с <see cref="NotFound"/>, то есть «трудоустройство не подтверждено».
|
||||
/// </summary>
|
||||
None = 0,
|
||||
|
||||
/// <summary>Сотрудника с таким ФИО в справочнике нет вовсе — трудоустройство не подтверждено.</summary>
|
||||
NotFound,
|
||||
NotFound = 1,
|
||||
|
||||
/// <summary>Работает хотя бы в одной компании.</summary>
|
||||
Employed,
|
||||
Employed = 2,
|
||||
|
||||
/// <summary>Есть в справочнике, но уволен из всех компаний, где числился.</summary>
|
||||
Dismissed,
|
||||
Dismissed = 3,
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
|
||||
@@ -1,8 +1,9 @@
|
||||
namespace Tnsu.ApprovalRoute.Application.Reminders;
|
||||
|
||||
/// <param name="ManagerEmail">
|
||||
/// То же значение, что и унаследованный ReceiverEmail, — email руководителя
|
||||
/// принимаемого сотрудника, кому адресовано напоминание.
|
||||
/// <param name="ReceiverEmail">
|
||||
/// Email руководителя принимаемого сотрудника — кому адресовано напоминание.
|
||||
/// Отдельного ManagerEmail рядом нет намеренно: это было то же самое значение
|
||||
/// под вторым именем.
|
||||
/// </param>
|
||||
/// <param name="CandidateEmail">Email принимаемого сотрудника (см. ReminderService.GetProbationEndingAsync).</param>
|
||||
/// <param name="CandidateName">ФИО принимаемого сотрудника (колонка CandidateName списка данных).</param>
|
||||
@@ -11,7 +12,7 @@ namespace Tnsu.ApprovalRoute.Application.Reminders;
|
||||
/// <param name="ProbationEndsAt">Дата окончания испытательного срока (HireDate + срок).</param>
|
||||
/// <param name="DaysUntilProbationEnds">Сколько дней осталось до конца испытательного срока.</param>
|
||||
public sealed record ProbationEndingReminderDto(
|
||||
string ManagerEmail,
|
||||
string ReceiverEmail,
|
||||
string CandidateEmail,
|
||||
string CandidateName,
|
||||
string? ProbationPeriod,
|
||||
@@ -20,4 +21,4 @@ public sealed record ProbationEndingReminderDto(
|
||||
int DaysUntilProbationEnds,
|
||||
string Url,
|
||||
string? ReceiverName = null)
|
||||
: ReminderDto(ManagerEmail, ReceiverName, Url);
|
||||
: ReminderDto(ReceiverEmail, ReceiverName, Url);
|
||||
|
||||
@@ -1,16 +1,15 @@
|
||||
namespace Tnsu.ApprovalRoute.Application.Reminders;
|
||||
|
||||
/// <summary>
|
||||
/// Какой вид напоминаний запросить у <see cref="ReminderService.GetRemindersAsync(DateTimeOffset, ReminderFeed, CancellationToken)"/>.
|
||||
/// Какой вид напоминаний запросить у <see cref="ReminderService.GetRemindersAsync(ReminderFeed, CancellationToken)"/>.
|
||||
///
|
||||
/// GetRemindersAsync изначально (до появления ProbationEnding) отдавал только
|
||||
/// просроченные шаги согласования — этот вид и остаётся значением по умолчанию
|
||||
/// (0), чтобы старые потребители (Power Automate), не указывающие feed явно,
|
||||
/// продолжали получать ровно то же, что и раньше.
|
||||
/// <see cref="None"/> («вид не указан») трактуется как <see cref="ApprovalOverdue"/>:
|
||||
/// до появления ProbationEnding эндпоинт отдавал только просроченные шаги, и
|
||||
/// вызов без явного выбора обязан работать как раньше.
|
||||
/// </summary>
|
||||
public enum ReminderFeed
|
||||
{
|
||||
None = 0,
|
||||
ApprovalOverdue = 1,
|
||||
ProbationEnding = 2
|
||||
ProbationEnding = 2,
|
||||
}
|
||||
|
||||
@@ -403,15 +403,20 @@ public sealed class ReminderService(
|
||||
var reminders = new List<ReminderDto>();
|
||||
foreach (var row in rows)
|
||||
{
|
||||
var reminder = await TryBuildProbationReminderAsync(type, row, listId, directory, now, cancellationToken);
|
||||
var reminder = await TryBuildProbationReminderAsync(row, directory, now, cancellationToken);
|
||||
if (reminder is not null) reminders.Add(reminder);
|
||||
}
|
||||
|
||||
return reminders;
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Ни type, ни listId здесь больше не нужны: ссылка в этом виде напоминаний —
|
||||
/// константа из настроек (см. ProbationReminderUrl), а поля самой заявки
|
||||
/// (RequestTypeId и т.п.) в ProbationEndingReminderDto не входят.
|
||||
/// </summary>
|
||||
private async Task<ReminderDto?> TryBuildProbationReminderAsync(
|
||||
RequestType type, HireRow row, string listId, EmployeeDirectorySnapshot directory,
|
||||
HireRow row, EmployeeDirectorySnapshot directory,
|
||||
DateTimeOffset now, CancellationToken cancellationToken)
|
||||
{
|
||||
if (string.IsNullOrWhiteSpace(row.CandidateName) || row.HireDate is not { } hireDate) return null;
|
||||
@@ -425,11 +430,17 @@ public sealed class ReminderService(
|
||||
|
||||
var candidateName = row.CandidateName.Trim();
|
||||
|
||||
// Перечислены оба «продолжающих» исхода явно, а «пропустить» — это default:
|
||||
// так любое НЕ подтверждённое трудоустройство (в том числе EmploymentStatus.None,
|
||||
// если он когда-нибудь сюда доедет) приводит к пропуску с предупреждением, а не
|
||||
// молча считается «работает».
|
||||
switch (directory.EmploymentOf(candidateName))
|
||||
{
|
||||
case EmploymentStatus.Employed:
|
||||
break;
|
||||
case EmploymentStatus.Dismissed:
|
||||
return null; // уже уволен — напоминать не о чем.
|
||||
case EmploymentStatus.NotFound:
|
||||
default:
|
||||
logger.LogWarning(
|
||||
"Заявка на приём {RequestId}: кандидат «{Fio}» не найден в /Employee/FullBase — статус трудоустройства не подтверждён, напоминание пропущено",
|
||||
row.Id, candidateName);
|
||||
@@ -455,26 +466,17 @@ public sealed class ReminderService(
|
||||
}
|
||||
|
||||
return new ProbationEndingReminderDto(
|
||||
ManagerEmail: managerEmail,
|
||||
ReceiverEmail: managerEmail,
|
||||
CandidateEmail: candidateEmail,
|
||||
CandidateName: candidateName,
|
||||
ProbationPeriod: row.ProbationPeriod,
|
||||
HireDate: hireDate,
|
||||
ProbationEndsAt: probationEndsAt,
|
||||
DaysUntilProbationEnds: daysLeft,
|
||||
Url: BuildProbationUrl(type, row, listId),
|
||||
// Не ссылка на саму заявку на приём (она уже закрыта, решать по ней
|
||||
// нечего) — константный адрес раздела, где руководитель заводит новую
|
||||
// заявку по итогам испытательного срока. См. ProbationReminderUrl.
|
||||
Url: options.Value.ProbationReminderUrl,
|
||||
ReceiverName: directory.NameByEmail(managerEmail));
|
||||
}
|
||||
|
||||
/// <summary>
|
||||
/// Ссылка на саму заявку (не на шаг согласования — этого вида напоминаний
|
||||
/// понятие "фаза" не касается), в том же формате, что и BuildUrl.
|
||||
/// </summary>
|
||||
private string BuildProbationUrl(RequestType type, HireRow row, string listId) =>
|
||||
options.Value.RequestUrlTemplate
|
||||
.Replace("{siteUrl}", type.SiteUrl!.TrimEnd('/'))
|
||||
.Replace("{pageUrl}", type.PageUrl)
|
||||
.Replace("{requestId}", row.Id.ToString())
|
||||
.Replace("{listId}", Uri.EscapeDataString(listId))
|
||||
.Replace("{nav}", "approvals");
|
||||
}
|
||||
|
||||
@@ -34,4 +34,19 @@ public sealed class RemindersOptions
|
||||
|
||||
/// <summary>За сколько дней до конца испытательного срока присылать напоминание.</summary>
|
||||
public int ProbationReminderDaysBeforeEnd { get; set; } = 3;
|
||||
|
||||
/// <summary>
|
||||
/// Куда ведёт ссылка в напоминании об окончании испытательного срока — на
|
||||
/// раздел «Подать заявку» портала, а НЕ на саму заявку на приём: от
|
||||
/// руководителя по этому напоминанию требуется завести НОВУЮ заявку (решение
|
||||
/// по итогам испытательного срока), а не открыть уже закрытую старую.
|
||||
///
|
||||
/// Поэтому ссылка константная, без подстановок: адрес конкретной формы в
|
||||
/// портал вообще не передать — параметр rptype без rpid/rpsource
|
||||
/// игнорируется, и маршрут сваливается в раздел «Подать заявку» (см.
|
||||
/// parseRoute в request-portal/shared/routing.ts). Отдельная настройка, а не
|
||||
/// хардкод, — чтобы адрес правился конфигурацией.
|
||||
/// </summary>
|
||||
public string ProbationReminderUrl { get; set; } =
|
||||
"https://tnsukz.sharepoint.com/sites/requests?rpnav=new";
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user