Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Content modelling/550 fix cache busting #9459

Merged
merged 3 commits into from
Sep 24, 2024
Merged

Content modelling/550 fix cache busting #9459

merged 3 commits into from
Sep 24, 2024

Commits on Sep 24, 2024

  1. return publishing app on HostContentItems 

    Now that this PR is merged [1] we should be able to retrieve the name of
the publishing app for Host Content. We will shortly use this when
creating Publish Intents to remove the cache when updating Host Documents.

[1] alphagov/publishing-api#2876
    @Harriethw
    Harriethw committed Sep 24, 2024
    Configuration menu
    Copy the full SHA
    0a57d49 View commit details
    Browse the repository at this point in the history

  2. add optional publishing_app param in PublishIntentPresenter 

    we want to send Publish Intents for Host Content (e.g. Editions that
are using embedded content/Content Blocks) [1] that is published
by other apps - this allows an optional app to be passed in, but retains
the default Whitehall value in order not to break existing functionality.

[1] #9447
    @Harriethw
    Harriethw committed Sep 24, 2024
    Configuration menu
    Copy the full SHA
    3895f0a View commit details
    Browse the repository at this point in the history

  3. pass publishing app name to Publish Intents for Host Content 

    We want to remove the cache Host Content that is not published by Whitehall,
as embedded content can be used across all Publishing apps.

[1] alphagov/publishing-api#2876
    @Harriethw
    Harriethw committed Sep 24, 2024
    Configuration menu
    Copy the full SHA
    d1c3824 View commit details
    Browse the repository at this point in the history