Replace the `dict` response in `load_existing_datasets` with a Pydantic class

[JackHopkins]

We should avoid passing loosely typed dictionaries around if we can avoid it. This should be replaced by a Pydantic class.

Note: This is in BufferedLogger.

[cosmastech]

Hi! I'm pretty new to Python and was looking for good first issues to tackle.

Was curious to know why you would use a Pydantic class here rather than a simple dataclass for this?

[JackHopkins]

Hey @cosmastech, a simple dataclass could work just as well!

If you're looking to contribute - awesome! I can create a couple of issues that are good starter issues to get to grips with the codebase.

I'll tag you in them when I've put them up.

[cosmastech]

Hey @cosmastech, a simple dataclass could work just as well!

If you're looking to contribute - awesome! I can create a couple of issues that are good starter issues to get to grips with the codebase.

I'll tag you in them when I've put them up.

Cool! Thanks.

[JackHopkins]

Could you please have a look at this one? @cosmastech