storybook.js.org

[kylegach]

• There is reasonable content on the page (see: No Content)
• I have read and accepted the Terms and Conditions

The content is at storybook-frontpage.netlify.app.

Per #9320, the test worked, so this PR makes the change on the real domain.

Making this a draft PR for now, to avoid premature merging.

@indus and I have coordinated to merge this on October 11, 2024 at 15:30 UTC.

[indus]

@kylegach I'm ready I make the change at 15:30 UTC.

[indus]

Just made change. Distribution to every node in the CF global network may take a minute.

[kylegach]

It's looking good! Thank you so much for your generous help, Stefan.

[indus]

@kylegach It's working for me.

May I ask you to change the PR according to this: https://github.com/js-org/js.org/actions/runs/11294549413/job/31416661395?pr=9321
(the CI expects netlify.app instead of netlify.com because of this ; I can keep it as .com in the zonefile if you prefer).

[indus]

As you have closed the other PR I will have to remove the storybook-test2 subdomain from the zonefile; to keep it in sync.

[kylegach]

May I ask you to change the PR according to this: js-org/js.org/actions/runs/11294549413/job/31416661395?pr=9321
(the CI expects netlify.app instead of netlify.com ; I can keep it as .com in the zonefile if you prefer).

Done!

As you have closed the other PR I will have to remove the storybook-test2 subdomain from the zonefile; to keep it in sync.

Yes, we would appreciate that. Thank you.

[indus]

should I try to make the .com -> .app change in the zonefile to keep it in perfect sync? ...or better leave it as is now that it works?

There is this comment: https://answers.netlify.com/t/deprecating-automatic-com-redirects/115442
But I'm really not sure this is also the new default for apex-loadbalancer.netlify(.com|.app)

[kylegach]

@indus — If it's alright, let's leave the zonefile as it is (using .com). Everywhere I found that apex-loadbalancer.netlify.com URL documented, it uses .com.

Do you have any sites configured using apex-loadbalancer.netlify.app?

[indus]

https://github.com/kylegach/js.org/blob/4805afb8a0d3d9711db231e508593011baaf32a5/cnames_active.js#L550 👍

[kylegach]

Oh. Then sure, let's keep it in sync while we're both actively monitoring and available. Thanks.

[indus]

Thanks. Just made the change to .app

[kylegach]

✅ All still looks good!