flutter_bloc_devtools icon indicating copy to clipboard operation
flutter_bloc_devtools copied to clipboard
verified publisher icon andrea689

[WIP] No mappable

Open KalilDev opened this issue 5 years ago • 2 comments

Simplified the logic a bit, got rid of Mappable, implemented the toJson replacement and migrated the example packages. I've yet to add tests, and i want to know what you think.

KalilDev avatar Dec 12 '20 23:12 KalilDev

Good job @KalilDev, thank you for contribution!!

I think we need to add _maybeToMap as well, for backwards compatibility and because who use quicktype it generate Map<String, dynamic> toMap() and not Map<String, dynamic> toJson()

andrea689 avatar Dec 21 '20 09:12 andrea689

Alright, totally agree! Im sorry, i was a bit busy this week and could not do it. Im going on a trip and when i come back, in the 28th, i will continue the work.

KalilDev avatar Dec 24 '20 11:12 KalilDev