-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs(fields): BelongsToMany updated ->inLine method, ModelResources->Buttons updated with new methods, Import/Export added sugar for ->withConfirm() method #524
Conversation
…modifyExportButton,getExportButton, modifyImportButton, getImportButton, modifyFiltersButton, getFiltersButton. Added images for these methods.
@DissNik немного обновил документацию, в данном ПР присутствуют изменения из ошибочно закрытого мною ПР. |
@@ -250,6 +250,114 @@ public function getMassDeleteButton( | |||
} // [tl! focus:-11] | |||
</x-code> | |||
|
|||
<x-sub-title id="mass-delete">Export button</x-sub-title> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
id не правильный, и стоит добавить в меню эти пункты
} // [tl! focus:-11] | ||
</x-code> | ||
|
||
<x-sub-title id="mass-delete">Import button</x-sub-title> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Тут тоже id нужно изменить чтоб из меню можно было перейти к этому разделу
@@ -250,6 +250,114 @@ public function getMassDeleteButton( | |||
} // [tl! focus:-11] | |||
</x-code> | |||
|
|||
<x-sub-title id="mass-delete">Кнопка экспорта</x-sub-title> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
аналогично
} // [tl! focus:-11] | ||
</x-code> | ||
|
||
<x-sub-title id="mass-delete">Кнопка импорта</x-sub-title> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
и тут
🤦 спасибо за замечания, не по глазам были id'шники ... поправил ru + en |
Сделал один небольшой ПР для обновления документации:
->inLine()
поля BelongsToMany;modifyExportButton
,getExportButton
,modifyImportButton
,getImportButton
,modifyFiltersButton
,getFiltersButton
в разделе ModelResources->Buttons;->withConfirm()
для ExportHandler.