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

fix(render): redirect to 200.html for prerender #9122

Merged
merged 1 commit into from
Sep 1, 2023

Conversation

jtoar
Copy link
Contributor

@jtoar jtoar commented Sep 1, 2023

@thedavidprice and I were seeing problems in deploy target CI related to CDNs using prerender. Netlify already has this 200.html file for prerender; using it render led to similar success. Original PR on 200.html for context: #4782

@jtoar jtoar added the release:fix This PR is a fix label Sep 1, 2023
@jtoar jtoar added this to the next-release-patch milestone Sep 1, 2023
@jtoar jtoar added the fixture-ok Override the test project fixture check label Sep 1, 2023
@jtoar jtoar enabled auto-merge (squash) September 1, 2023 18:17
@jtoar jtoar merged commit c3f045c into main Sep 1, 2023
30 of 31 checks passed
@jtoar jtoar deleted the ds-render/redirect-to-200.html-for-prerender branch September 1, 2023 18:18
jtoar added a commit that referenced this pull request Sep 2, 2023
@thedavidprice and I were seeing problems in deploy target CI related to
CDNs using prerender. Netlify already has this 200.html file for
prerender; using it render led to similar success. Original PR on
200.html for context:
[#4782
@jtoar jtoar modified the milestones: next-release-patch, v6.1.1 Sep 2, 2023
jtoar added a commit that referenced this pull request Sep 2, 2023
@thedavidprice and I were seeing problems in deploy target CI related to
CDNs using prerender. Netlify already has this 200.html file for
prerender; using it render led to similar success. Original PR on
200.html for context:
[#4782
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fixture-ok Override the test project fixture check release:fix This PR is a fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants