Support for voc #14
Support for voc #14
Conversation
src/utils/ExportUtil.ts
Outdated
|
|
||
| export class ExportUtil { | ||
| public static getProjectName(): string { | ||
| return store.getState().editor.projectName |
There was a problem hiding this comment.
ja bym czegoś takiego użył:
https://www.npmjs.com/package/xml-escape
There was a problem hiding this comment.
i w sumie fajnie że to robisz... ale imho do reducera powinno pójść już elegancko wyESCAPEowana wartość... czyli to sanitizowanie powinno być na onChange albo onBlur
There was a problem hiding this comment.
albo przy wykonywaniu akcji
src/utils/ExportUtil.ts
Outdated
| @@ -0,0 +1,14 @@ | |||
| import {store} from ".."; | |||
|
|
|||
| export class ExportUtil { | |||
There was a problem hiding this comment.
w Reduxie jest taka konwencja, że takie obiekty jak ten, nazywają się SELECTORAMI,
czyli w Twoim przypadku byłby to jakiś selector np: EditorSelector.ts (ponieważ pobierasz dane z reducera "editor")
i ma w sobie taką samą metodę jak teraz masz... czyli zmień tylko nazwę na "EditorSelector.ts"
There was a problem hiding this comment.
generalnie poczytaj o selektorach
No description provided.