Если имя персонажа в непринятой заявке в слоте изменено на другое, при принятии заявки замещение имени персонажа-Слота не происходит

[tsareva]

Насколько я помню, изначально у нас считалось, что предварительно заполненное мастером более важно, чем заполненное игроком; но кажется, что для Слотов это может быть не так (по крайней мере, в имени персонажа точно)

[leotsarev]

По моей логике заполненное игроком важнее (не нравится, заявку не принимай). Но как работает я не помню

[tsareva]

Да, для обычных заявок работает так, как ты сказал (при принятии заявки происходит оверрайд заполненного игроком). Кстати, это может быть или в явном виде где-то написать при принятии (что после принятия заявки такие-то заполненные поля будут замещены на такие-то значения)? При том, что история изменений персонажа не хранится, может быть неприятный сюрприз.

А для слотов сохраняется имя Слота, что очевидно неверно, должно происходить замещение на имя от игрока.

[tsareva]

Как быть, кстати, с тз формы заявки, если мы хотим, чтобы слотам имена предлагались, а поименованным в сетке нет?
Можно, конечно, писать описание поля «оставьте пустым, если заявляетесь на персонажа с именем в сетке», но кажется, что это костыль и у полей заявки должно быть свойство «показывать для заполнения только слотам». Что думаешь?

[leotsarev]

Да, для обычных заявок работает так, как ты сказал (при принятии заявки происходит оверрайд заполненного игроком).

По коду выглядит, что ПЕРСОНАЖ важнее. Это неправильно, должна быть ЗАЯВКА важнее. Согласна?

[tsareva]

Я согласна, но это должно быть явно видно в ui. Или будет можно случайно затереть мастерское, которое нигде не сохранено в придачу чт, 14 окт. 2021 г., 19:34 Leonid Tsarev ***@***.***>:

…

Да, для обычных заявок работает так, как ты сказал (при принятии заявки происходит оверрайд заполненного игроком). По коду выглядит, что ПЕРСОНАЖ важнее. Это неправильно, должна быть ЗАЯВКА важнее. Согласна? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#1549 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ACB53FEQ3NRYORMGSV33L3LUG4BBJANCNFSM5FRAYX3Q> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[leotsarev]

Вопрос — нужно ли что-то тут менять перед релизом слотов?