magento / devdocs

[Deprecated] Magento Developer Documentation
Open Software License 3.0
674 stars 1.76k forks source link

The procedure for managing domains on the project is partially incorrect #9522

Closed ak77-prime closed 1 year ago

ak77-prime commented 2 years ago

Is there an existing issue for this?

Which topic?

https://devdocs.magento.com/cloud/cdn/configure-fastly-customize-cache.html#manage-domains

What's wrong with the content?

For Starter projects, use the Domains option to add and manage the Fastly domain configuration for your store. Before adding a domain to your project, you must submit a Adobe Commerce Support ticket to add the domain to your Cloud configuration. You can add the domain to Fastly after Magento confirms your request.

For Pro plan projects, you must submit a Adobe Commerce support ticket to update the Fastly domain configuration for your project.

The above is only partially correct. The steps below that begin with To manage Fastly domain configuration from the Admin apply to both Starter and Pro - but Pro projects must submit a request to complete the Nginx configuration.

In addition, Step 5 is incorrect:

You can only specify domains that have already been added to your Cloud infrastructure configuration.

What changes do you propose?

For both Starter and Pro projects, you can use the Domains option to add and manage the Fastly domain configuration for your store.

To manage Fastly domain configuration from the Admin: ... Step 5 - if the pre-existing domain(s) has been configured on a different Fastly account, you must submit a ticket to request for Domain Delegation before you can proceed with specifying the domains.

Anything else that can help to cover this?

No response

m2-assistant[bot] commented 2 years ago

Hi @ak77-prime. Thank you for your report. To speed up processing of this issue, make sure that you provided sufficient information.

Add a comment to assign the issue: @magento I am working on this


dobooth commented 1 year ago

@hguthrie Something you want to handle?

hguthrie commented 1 year ago

I think this was addressed last month. @meker12 can we close this?

meker12 commented 1 year ago

This update has been added to the current documentation on ExL. We can close this ticket.