[Promises, Part 1] TGUI Admin tickets #13631
Conversation
Changelog status: ✔️ |
AzzyDreemurr13
added
the
Admin
| if("legacy") | ||
| TicketPanelLegacy() |
There was a problem hiding this comment.
если и оставлять легаси-уи то мб в префах это сохранять
There was a problem hiding this comment.
Ну тут на вкус и цвет, не думаю что критично. Подожду мнения других мейнтеров пока что
| </LabeledList.Item> | ||
| )} | ||
| <LabeledList.Item label="Actions"> | ||
| <div dangerouslySetInnerHTML={{ __html: actions }} /> |
There was a problem hiding this comment.
а в чем смысл тгуи-обертки если мы по-прежнему пользуемся сомнительным dangerouslySetInnerHTML?
There was a problem hiding this comment.
У тебя есть предложения как это можно реализовать лучше?
There was a problem hiding this comment.
реализовать то как и все остальные тгуи интерфейсы
без dangerouslySetInnerHTML
у тебя буквально в этом же интерфейсе используются тгуишные кнопки
There was a problem hiding this comment.
dangerouslySetInnerHTML Запрещен, мы же тебе еще в прошлый раз указывали и говорили как без него
There was a problem hiding this comment.
Спасибо вам обоим, а Киборгу за подсказку из прошлого. (Исправил)
|
И небольшой факт, так как ТГУИ, то теперь не надо будет прожимать по КД рефрешь, все моментально появится в тикете |
Co-authored-by: NinjaPikachuska <[email protected]>
| if(..()) | ||
| return | ||
|
|
||
| TicketPanelLegacy() |
There was a problem hiding this comment.
Для вызова старой панельки тикетов в случае чего, есть предложения куда переместить?
Спасибо Киборгу за подсказки из прошлого Co-Authored-By: KIBORG04 <[email protected]>
| > | ||
| <LabeledList> | ||
| <LabeledList.Item label="Admin Help Ticket"> | ||
| #{id}: <div dangerouslySetInnerHTML={{ __html: name }} /> |
There was a problem hiding this comment.
dangerouslySetInnerHTML до сих пор используется
| </LabeledList.Item> | ||
| )} | ||
| <LabeledList.Item label="Actions"> | ||
| {actions} |
There was a problem hiding this comment.
ты все еще пытаешься хтмл-кнопки запихнуть в тгуи, только теперь он даже не будет воспринимать эту разметку (посмотри на локалке хоть что получилось)
у тебя в этом же интерфейсе используются тгуи-кнопки
<Button icon="pen" onClick={() => act('retitle')}>
Rename Ticket
</Button>и по такой аналогии все кнопки должны быть
There was a problem hiding this comment.
Ммм, понял тебя, спасибо Пикачушка
| </LabeledList.Item> | ||
| <LabeledList.Item label="Log"> | ||
| {Object.keys(log).map((L, i) => ( | ||
| log[L] |
There was a problem hiding this comment.
то же самое, логи уже обернуты в теги, которые опять же теперь еще и не будут восприниматься
Описание изменений
Как-то раз пока был Админом, я в ЦК заикнулся про ТГУИ тикеты. В общем, пока ещё не успел уйти в одно место, начинаю выпускать партию ПРов в которых я обещал что-то сделать. Один из них ТГУИ панелька ahelp!
Почему и что этот ПР улучшит
Переносим интерфейсы на ТГУИ формат по плану переносов всех UI на ТГУИ
Авторство
Моё (портировано с VOREStation/VOREStation#13845)
Чеинжлог
🆑