Skip to content

Handle special case of Pilot for Hosting Policy - #1159

Merged
dati18 merged 1 commit into
T432965from
T432965-handle-pilot-footer
Jul 29, 2026
Merged

Handle special case of Pilot for Hosting Policy#1159
dati18 merged 1 commit into
T432965from
T432965-handle-pilot-footer

Conversation

@tarrow

@tarrow tarrow commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

A rather unelegant workaround to handle the case of no currrent hosting policy to redirect people to the pilot.

This allows us to change the footer link to be just '/hosting-policy' and have it handled by the HostingPolicyRenderer but still redirect to the pilot component.

Added comments to ensure we remove this cruft as soon as the pilot policy is redundant.

Bug: T432965

A rather unelegant workaround to handle the case of no currrent hosting
policy to redirect people to the pilot.

This allows us to change the footer link to be just '/hosting-policy'
and have it handled by the HostingPolicyRenderer but still redirect
to the pilot component.

Added comments to ensure we remove this cruft as soon as the pilot policy
is redundant.

Bug: T432965
@github-actions

Copy link
Copy Markdown

Deployment previews on netlify for branch refs/pull/1159/merge will be at the following locations (when build is done):

@tarrow
tarrow marked this pull request as ready for review July 29, 2026 11:59
@dati18
dati18 merged commit 296c72a into T432965 Jul 29, 2026
7 checks passed
@dati18
dati18 deleted the T432965-handle-pilot-footer branch July 29, 2026 12:04
dati18 pushed a commit that referenced this pull request Jul 29, 2026
A rather unelegant workaround to handle the case of no currrent hosting
policy to redirect people to the pilot.

This allows us to change the footer link to be just '/hosting-policy'
and have it handled by the HostingPolicyRenderer but still redirect to
the pilot component.

Added comments to ensure we remove this cruft as soon as the pilot
policy is redundant.

Bug: T432965
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants