Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to fapi-client v4.0.6 #26578

Merged
merged 1 commit into from
Sep 13, 2023
Merged

Update to fapi-client v4.0.6 #26578

merged 1 commit into from
Sep 13, 2023

Conversation

rtyley
Copy link
Member

@rtyley rtyley commented Sep 13, 2023

This is a small upgrade, catching up with the recent dependency updates of https://github.com/guardian/facia-scala-client/releases/tag/v4.0.6, before the more extensive update in guardian/facia-scala-client#287 is introduced.

This update has already been tested in Ophan's PromotionPoller with https://github.com/guardian/ophan/pull/5540, successfully deployed to Prodution.

Note that as https://github.com/guardian/frontend is already using Play 2.8, it should probably be using fapi-client-play28, rather than fapi-client-play27, so I've also updated that.

This is a small upgrade, catching up with the recent dependency updates of
https://github.com/guardian/facia-scala-client/releases/tag/v4.0.6,
before the more extensive update in guardian/facia-scala-client#287
is introduced.

This update has already been tested in Ophan's PromotionPoller with
guardian/ophan#5540, successfully deployed
to Prodution.

Note that as https://github.com/guardian/frontend is _already_
using Play 2.8, it should probably be using `fapi-client-play28`,
rather than `fapi-client-play27`, so I've also updated that.
@rtyley rtyley requested a review from a team as a code owner September 13, 2023 15:24
Copy link
Contributor

@ioannakok ioannakok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @rtyley!

@prout-bot
Copy link
Collaborator

Seen on ADMIN-PROD (merged by @rtyley 18 minutes and 5 seconds ago)

@prout-bot
Copy link
Collaborator

Seen on FRONTS-PROD (merged by @rtyley 18 minutes and 13 seconds ago)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants