Skip to content

[CRUD] Support non-editable items in form pages#4918

Merged
apedroferreira merged 5 commits intomui:masterfrom
apedroferreira:crud-whitelist
May 22, 2025
Merged

[CRUD] Support non-editable items in form pages#4918
apedroferreira merged 5 commits intomui:masterfrom
apedroferreira:crud-whitelist

Conversation

@apedroferreira
Copy link
Collaborator

@apedroferreira apedroferreira commented May 1, 2025

Closes #4848 by leveraging the editable property in DataGrid fields.
Makes it so some fields can be hidden in form pages.

https://deploy-preview-4918--mui-toolpad-docs.netlify.app/toolpad/core/react-crud/#non-editable-fields

@apedroferreira apedroferreira added type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature. scope: toolpad-core Abbreviated to "core" component: crud labels May 1, 2025
@apedroferreira apedroferreira self-assigned this May 1, 2025
@apedroferreira apedroferreira changed the title Support non-editable items in form pages [CRUD] Support non-editable items in form pages May 1, 2025
@mui-bot
Copy link

mui-bot commented May 1, 2025

Netlify deploy preview

https://deploy-preview-4918--mui-toolpad-docs.netlify.app/

Generated by 🚫 dangerJS against 09d34e6

@apedroferreira apedroferreira moved this to In progress in Toolpad public roadmap May 6, 2025
@apedroferreira apedroferreira marked this pull request as ready for review May 6, 2025 17:20
@apedroferreira apedroferreira requested a review from a team May 6, 2025 17:20
@apedroferreira apedroferreira merged commit 9275a90 into mui:master May 22, 2025
15 checks passed
@apedroferreira apedroferreira deleted the crud-whitelist branch May 22, 2025 12:57

const newNote = {
id: notesStore.reduce((max, note) => Math.max(max, note.id), 0) + 1,
createdAt: new Date().toISOString(),
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

In the docs example this createdAt doesn't seem to be working... Will fix separately now that I already merged it.

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

Labels

scope: toolpad-core Abbreviated to "core" type: enhancement It’s an improvement, but we can’t make up our mind whether it's a bug fix or a new feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CRUD] Fields whitelist

3 participants