DataClass toJson/fromJson with column names about drift

[davidmartos96]

@simolus3 Yes, I definitely agree, and it adds unneccesary complexity to the drift package. Custom row classes is a nice solution to give more control to the user.
This was just asking for some way to configure it globally. In any case, we'll add an option to annotate with UseRowClass if the user needs to.
Thanks!

[simolus3]

It is a pretty old option. I think we can replace it with a newer one applying to both kinds of table declarations. That said, you can also do this in electricsql when generating drift tables by adding a @JsonKey annotation to the column (the annotation is from drift, not the one from json_serializable). Since you're already generating tables, an alternative might be to generate row classes with a @JsonSerializable annotation as well and then add a @UseRowClass annotation to generated tables.

IMO, having JSON serialization built into drift is a mistake. That feature was part of the very first version and now I can't really remove it anymore. But serialization and database access are different problems, and ideally drift shouldn't have to deal with JSONs at all. The configuration options drift has to customize serialization properties will always lack behind what json_serializable and others have to offer.