guardian / facia-scala-client

Low level client for the Facia JSON API
Other
2 stars 1 forks source link

Update default collection type #316

Closed cemms1 closed 3 months ago

cemms1 commented 3 months ago

What does this change?

Swaps "fixed/small/slow-VI" for "fixed/small/slow-IV"

Why

The container type "fixed/small/slow-VI" does not exist, so it looks like this might be a mistake. MAPI updated the equivalent logic in 2017: https://github.com/guardian/mobile-apps-api/pull/1038

Though, I'm not even sure this is ever used! The git blame says this was last updated about 10 years ago 😵 (https://github.com/guardian/facia-scala-client/pull/31)