Skip to content

Conversation

@davidhariri
Copy link
Contributor

@davidhariri davidhariri commented May 1, 2022

Description


Before submitting the PR, please make sure you do the following:

  • Read the Contributing Guidelines
  • Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123)
  • Update docs in this repository according to your change
  • Run yarn changeset add if this PR cause a version bump based on Keep a Changelog and adheres to Semantic Versioning

@davidhariri davidhariri changed the title Resolves #1087 Adds preconnect <link> for CDN May 1, 2022
@davidhariri davidhariri marked this pull request as ready for review May 1, 2022 20:40
@davidhariri
Copy link
Contributor Author

Please note that I did not test this change locally before opening this PR. Does this repo have some automated tests?

@jplhomer jplhomer requested a review from a team May 2, 2022 16:37
benjaminsehl
benjaminsehl approved these changes May 2, 2022
Copy link
Member

@benjaminsehl benjaminsehl left a comment

Choose a reason for hiding this comment

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

Hello World starter is complete, but we also need this on the Demo Store starter here: https://github.com/Shopify/hydrogen/blob/v1.x-2022-07/examples/template-hydrogen-default/index.html

@benjaminsehl benjaminsehl merged commit d12c1f7 into Shopify:v1.x-2022-07 May 2, 2022
blittle added a commit that referenced this pull request May 4, 2022
* v1.x-2022-07: (95 commits)
  [ci] release v1.x-2022-07 (#1170)
  Try ignoring hello-world to see if it will get bumped
  Don't consider examples part of the workspace (#1202)
  Fix headers on oxygen (#1201)
  Add bot user agents for Seoradar and Adresults, resolves #1199 (#1200)
  Fix changeset
  updates to docker deploy documentation to resolve run issues (#1196)
  Upgrade body-parser (#1162)
  Fix path for deployments
  Adds ability to add more than one cookie per response (#1161)
  Move Demo Store to templates folder (#1132)
  Avoid additional div element (#1191)
  Whoops this should only be patch
  Adds preconnect <link> for CDN (#1160)
  Bump ejs from 3.1.6 to 3.1.7 (#1147)
  Fix scroll restoration when server props are changed (#1152)
  Typo
  Fixes #1165 by making a missing alt tag a console warning (#1167)
  Remove concurrency directive for Oxygen deployments
  Fix hydrogen-ui dev and build issues (#1169)
  ...
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.

Add rel=preconnect tag for cdn.shopify.com

2 participants