-
Notifications
You must be signed in to change notification settings - Fork 375
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
Feature - Field translated text #1505
Feature - Field translated text #1505
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Thanks @lguichard I just ran into this when trying to edit a product |
@lguichard can you fix the conflicts and confirm whether this is now ready to review, as @alecritson mentioned there were some issues with the richtext editor? |
@glennjacobs Ok, i'll merge from 1.x soon. I tried to reproduce many times, i can't trigger the bug on Rich Editor 😄 I suspect bug with SPA mode with Filament. |
@glennjacobs That's it ! |
We're casting the name attribute so this is interfering and causing the name to not update.
We're casting the label attribute as name attribute
* 1.x: Feature - Discount editing (lunarphp#1542) Update ManageProductShipping.php Feature - Prevent delete action on resources with childs (lunarphp#1532) Feature - Brand urls and tweaks (lunarphp#1541) Chore - Update larastan/larastan (lunarphp#1558) chore: fix code style Feature - Field translated text (lunarphp#1505) Fix tests Revert "Quantity break fixes" Quantity break fixes Feature - Pricing management (lunarphp#1515) Improved dashboard KPIs
This PR fix definitely the TranslatedText component for the MVP.
It's a same UX approach that legacy version. Let's get a simple solution working and consider a better solution later.
The issue related to #1425