support for JSON fields

[kubamarkiewicz]

Hi, I was wondering if you are planning to support translating JSON fields?
In Spatie documentation I can see, that they support it: https://github.com/spatie/laravel-translatable

However, when I try it with your plugin, it doesn't work as expected.

Here is my Page model:

class Page extends Model
{
    use HasTranslations;

    protected $fillable = [
        'name',
        'date',
        'my_fields',
    ];

    protected $casts = [
        'my_fields' => 'json',
    ];

    public $translatable = ['name', 'my_fields->title'];

and PageResource:

    public static function form(Form $form): Form
    {
        return $form
            ->schema([
                TextInput::make('name'),
                DatePicker::make('date'),
                TextInput::make('my_fields.title'),
                TextInput::make('my_fields.slug'),

When I edit a record and save the data it seems to work, but when I refresh the page I get Title: {"es":""}:

[atmonshi]

can you test my_fields instead of my_fields->title in $translatable?

[kubamarkiewicz]

If I use public $translatable = ['name', 'my_fields']; then it works, but not as expected, because then it translates also my_fields->slug field, that should not be translated.

In the database it is saved as:
{"en": {"slug": "slug en", "title": "title en"}, "es": {"slug": "slug es", "title": "title es"}}
and should be:
{"slug": "slug", "title": {"en: "title en", "es": "title es"}}

[kubamarkiewicz]

Actually, the error is different, and it occurs on saving the data:

[atmonshi]

can you add the scenario to the demo so it easy to test and debug it
https://github.com/lara-zeus/translatable-demo
you can use any table or create a new one. and create a page to test

[kubamarkiewicz]

ok, I will add it the demo

[kubamarkiewicz]

hi, I have created the scenario for testing json fields:
lara-zeus/translatable-demo@main...kubamarkiewicz:translatable-demo:main
I have added json_fields column to books table and created a section in book edit form:

[atmonshi]

-did you forget to push the changes or create a PR :)-
ok didnt see the link sorry