Vtrushin/feature/sh-119 #1852
Vtrushin/feature/sh-119 #1852
Conversation
VasiliyTrue
force-pushed
the
vtrushin/feature/SH-119
branch
from
b1cf95b to
ebf9089
Compare
| const h = parts.length > 4 && parts[4] ? Number(parts[4]) : 1; | ||
|
|
||
| return { | ||
| i: `${groupKey}::${id}`, |
There was a problem hiding this comment.
такой формат идентификатора имеет какой-то смысл?
There was a problem hiding this comment.
ты про что конкретно? если про двойное двоеточие, то это универсальный паттерн для разделения неймспейсов, что бы избежать случаи, если одно двоеточие уже присутствует присутсвует в общей строке, например в groupKey или id
There was a problem hiding this comment.
я про то зачем groupKey в свойстве объекте где groupKey уже указан и так, при этом в закодированном виде как часть строки
| @@ -78,3 +79,54 @@ export const toGroupInfoPrefix = (i: number | string) => `${GET_PARAMS.dashboard | |||
| export const toPlotPrefix = (i: number | string) => (i && i !== '0' ? `${GET_PARAMS.plotPrefix}${i}.` : ''); | |||
| export const toVariablePrefix = (i: number | string) => `${GET_PARAMS.variablePrefix}${i}.`; | |||
| export const toVariableValuePrefix = (name: string) => `${GET_PARAMS.variableValuePrefix}.${name}`; | |||
|
|
|||
| // Compresses layout data into a compact string for URL | |||
| // Format: "id.x.y.w.h-id.x.y.w.h-..." where values are in sequential order | |||
There was a problem hiding this comment.
пока не очень нравиться предложенный формат хранения, хотелось бы обсудить данный вопрос
There was a problem hiding this comment.
давай созвонимся
| <PlotView | ||
| className={cn( | ||
| isDashboardEditAllowed && css.pointerEventsNone, | ||
| isNotMobile && 'position-relative overflow-hidden w-100 h-100' |
There was a problem hiding this comment.
условились в чате что оставляем, пока в твоей таске с виджетами не поправишь
VasiliyTrue
force-pushed
the
vtrushin/feature/SH-119
branch
from
882291f to
1a1fe53
Compare
No description provided.