Skip to content
This repository was archived by the owner on Nov 19, 2024. It is now read-only.

Removed outdated info about installing Fastly #5025

Merged
merged 19 commits into from
Jul 27, 2019

Conversation

meker12
Copy link
Contributor

@meker12 meker12 commented Jul 24, 2019

Purpose of this pull request

On Cloud Staging and Production environments, the Fastly module is installed automatically. Cloud Guide still says customer must install module on Integration branch.

Updates

  • Removed info about installing Fastly module through Integration branch, which is no longer required
  • Updated language, section organization, and other editorial changes to make Fastly module Set up instructions easier to understand
  • Corrected Backend settings configuration and added information about using Fastly Edge modules to add additional backends.
  • Cleaned up Fastly module upgrade instructions

Affected DevDocs pages

https://devdocs.magento.com/guides/v2.3/cloud/cdn/configure-fastly.html
https://devdocs.magento.com/guides/v2.3/cloud/cdn/cloud-fastly.html
https://devdocs.magento.com/guides/v2.3/cloud/cdn/trouble-fastly.html

whatsnew
Updated the Fastly Set up instructions to remove the requirement to install the Fastly CDN module for Magento 2 in Magento Commerce Cloud environments. The latest version of the Fastly module is installed by default in Pro Staging and Production environments and in the Starter Production (master) environment.

meker12 added 3 commits July 23, 2019 21:11
-  Added info about shared Fastly service account used for Magento Commerce Cloud projects
- Move info about domain delegation when you already have the Cloud domain and subdomain
associated to an existing Fastly account
- Moved info about Fastly service account to intro
- Moved domain delegation instructions to intro topic 
since domain delegation request must be completed before
Cloud projects can be added to the service account
- On Cloud Staging and Production environments, the Fastly module is installed automatically
- Removed info about installing through Integration branch
Copy link
Contributor

@hguthrie hguthrie left a comment

Choose a reason for hiding this comment

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

This is much improved! Much easier to read.

meker12 and others added 5 commits July 25, 2019 14:22
- Remove content that suggests that you can configure multiple backends from the Magento Admin
- Explain that the default Fastly VCL code provides default backend settings for review and update
- Add note about integrating additional backends using Fastly Edge modules.
- Verify default Fastly config before customizing settings or enabling advanced features
- Update the configuration checklist in topic overview to remove Fastly Image optimization
Copy link
Contributor

@hguthrie hguthrie left a comment

Choose a reason for hiding this comment

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

I spotted a couple of minor bits, but otherwise, Yay!

{:#info .bs-callout .bs-callout-info}
This does not work for an [apex domain](https://blog.cloudflare.com/zone-apex-naked-domain-root-domain-cname-supp) (also referred to as a _naked_ domain). You must use a DNS
provider that supports forwarding DNS queries to use an apex domain.
You need make the following changes to the DNS configuration for your site:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
You need make the following changes to the DNS configuration for your site:
You need to make the following changes to the DNS configuration for your site:

@meker12 meker12 changed the base branch from mae-fastly-setup-improvements to master July 26, 2019 19:24
meker12 added 2 commits July 26, 2019 14:28
- Fix broken links
- Editorial clean up
- Fixed broken link
- Cleaned up upgrade topic
@meker12 meker12 added Internal Dev Differentiates work between community and Magento staff Major Update Significant original updates to existing content 2.x labels Jul 26, 2019
{:#info .bs-callout .bs-callout-info}
This does not work for an [apex domain](https://blog.cloudflare.com/zone-apex-naked-domain-root-domain-cname-supp) (also referred to as a _naked_ domain). You must use a DNS
provider that supports forwarding DNS queries to use an apex domain.
To enable Fastly caching on your Staging and Production sites, you need make the following changes to the DNS configuration for your site:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
To enable Fastly caching on your Staging and Production sites, you need make the following changes to the DNS configuration for your site:
To enable Fastly caching on your Staging and Production sites, you need to make the following changes to the DNS configuration for your site:

@meker12
Copy link
Contributor Author

meker12 commented Jul 27, 2019

running tests

@meker12
Copy link
Contributor Author

meker12 commented Jul 27, 2019

running tests

@meker12
Copy link
Contributor Author

meker12 commented Jul 27, 2019

running tests

@meker12 meker12 merged commit 9e29ee2 into master Jul 27, 2019
@ghost
Copy link

ghost commented Jul 27, 2019

Hi @meker12, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

@meker12 meker12 deleted the mae-MAGECLOUD-3690-Fastly-module-auto-installed branch July 27, 2019 22:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
2.x Internal Dev Differentiates work between community and Magento staff Major Update Significant original updates to existing content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants