Skip to content

Conversation

@plyhun
Copy link
Contributor

@plyhun plyhun commented Jan 9, 2026

No description provided.

@plyhun plyhun requested a review from deckdom January 9, 2026 15:54
itemType = itemType || (item).type;
itemId = itemId === undefined ? item.id : itemId;
const parentPerms$ = this.forFolder(parentFolderId, nodeId);
const folderId = itemType === 'folder' ? itemId : this.getParentFolderId(item);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Main func change.

const CHARS_REQUIRE_ESCAPE = new Set([ '\\', '/', '{', '}', '[', ']', '(', ')', '.', '^', '$', '*', '+', '-', '?', '!', '=' ]);

const CONTROLS_I18N: (keyof EditableFolderProps)[] = ['nameI18n', 'descriptionI18n', 'publishDirI18n'];
const CONTROLS: (keyof EditableFolderProps)[] = ['name', 'description', 'publishDir'];
Copy link
Contributor Author

@plyhun plyhun Jan 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if I need to add extra checks for the existence of the separate (default / i18n) fields sets. There is a null check in the forms core lib though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants