dart-lang / site-www

Source for Dart website
https://dart.dev
Other
960 stars 693 forks source link

Update or remove 'Converters and codecs' article #5175

Open parlough opened 1 year ago

parlough commented 1 year ago

Page URL

https://dart.dev/articles/archive/converters-and-codecs

Page source

https://github.com/dart-lang/site-www/tree/main/src/_articles/archive/converters-and-codecs.md

Describe the problem

The article has been marked as obsolete for a long time, but still is hosted and provides valuable information.

Expected fix

Either remove the article or update it for modern standards:

parlough commented 5 months ago

@lrhn This old Converters and codecs article has been sitting, marked as obsolete, on the site for a long time now with outdated samples and perhaps inaccurate text.

Do you think you could take a quick look to see if it's worth me updating and cleaning it up? Or better to just drop in its current state?

lrhn commented 5 months ago

A quick look at the article doesn't show anything particularly wrong. The JsonCodec part mentions the toEncodable parameter, which I think was the most recent change, so it's probably up-to-date.

It's not a library that has had many changes. I think there are places where I'd phrase things differently, or maybe focus differently, but the actual content is fine. It's not because anything is outdated.

Are you aware of any particular issues with the content?