Skip to content

[docs] Remove relience of abbreviations#14226

Merged
oliviertassinari merged 1 commit intomui:masterfrom
oliviertassinari:e-evenet
Aug 16, 2024
Merged

[docs] Remove relience of abbreviations#14226
oliviertassinari merged 1 commit intomui:masterfrom
oliviertassinari:e-evenet

Conversation

@oliviertassinari
Copy link
Member

@oliviertassinari oliviertassinari commented Aug 15, 2024

This is a general guideline we have in the codebase, we avoid variable abbreviations so noobies can better understand what variable they are manipulating. Now, there is also value in having this always the same it's more intuitive.

I have noticed this from this tweet: https://x.com/ericclemmons/status/1823108565285941266, which could suggest that we should have TanStack Query integration demos. Issue created: #14227.

@oliviertassinari oliviertassinari added docs Improvements or additions to the documentation. scope: data grid Changes related to the data grid. dx labels Aug 15, 2024
@mui-bot
Copy link

mui-bot commented Aug 15, 2024

Deploy preview: https://deploy-preview-14226--material-ui-x.netlify.app/

Generated by 🚫 dangerJS against 4d019f3

@oliviertassinari oliviertassinari merged commit e557ed5 into mui:master Aug 16, 2024
@oliviertassinari oliviertassinari deleted the e-evenet branch August 16, 2024 09:45
@oliviertassinari oliviertassinari added the type: regression A bug, but worse, it used to behave as expected. label Aug 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to the documentation. scope: data grid Changes related to the data grid. type: regression A bug, but worse, it used to behave as expected.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants