Запрос на слияние из feature/ERA-551-excel-creator-plugin в dev
#82
Слито
3 месяца назад
- AОставлен комментарий
Нет содержимого
Обновлено только что - zaznobins
Рассмотрены изменения3 месяца назад - zaznobins
Рассмотрены изменения3 месяца назад - zaznobins
Рассмотрены изменения3 месяца назад - zaznobins
Запрос на слияние одобрен3 месяца назад - clay.zenx
Запрос на слияние одобрен3 месяца назад - 3 месяца назад
- 3 месяца назад
export class Book { constructor(ExcelJS, options = {}) {this.workbook = new ExcelJS.Workbook();
Тут больше оформительский такой момент. Немного странно делать класс зависимым от структуры ExcelJS, когда мы на деле используем только один параметр из неё. Логичнее будет сразу этот параметр (Workbook) в конструктор и пересылать - он будет более универсальным. Еще можно рассмотреть вариант посыла в конструктор сразу инстанса (const book = new Book(new Workbook(), options)). Тогда можно будет применить такой класс для создание нового Book с каким-нибудь модифицированным workbook, если вдруг будет нужно.
Обновлено 3 месяца назад
// // Первая колонка
// } else if (c === tableWidth - 1) {// // Последняя колонка
// }
Этот блок комментов точно намеренно тут оставлен?
Обновлено 3 месяца назад
components.forEach(component => { if (component.type === 'template') { if (!this.profile?.templates?.[component.name]) {Ну тут уж можно не чейнить this.profile. ) Фигня, но чуток рвёт шаблон при просмотре. На 107 то же.
Обновлено 3 месяца назад
Ревьюеры
Z
C