Skip to content

Conversation

@oredavids
Copy link
Contributor

@oredavids oredavids commented Jan 4, 2022

What does this PR do?

  • Adds site URL config. This fixes the /sitemap.xml page entries which had the wrong base URL

@netlify
Copy link

netlify bot commented Jan 5, 2022

✔️ Deploy Preview for pensive-meitner-faaeee ready!

🔨 Explore the source changes: e592624

🔍 Inspect the deploy log: https://app.netlify.com/sites/pensive-meitner-faaeee/deploys/61d5d4fafe651c0007013760

😎 Browse the preview: https://deploy-preview-126--pensive-meitner-faaeee.netlify.app

eak12913
eak12913 previously approved these changes Jan 5, 2022
Copy link
Contributor

@eak12913 eak12913 left a comment

Choose a reason for hiding this comment

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

LGTM.

Update the config files that push to stage and prod to set the proper SITE_URL
Copy link
Contributor

@ebeneliason ebeneliason left a comment

Choose a reason for hiding this comment

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

Stamp.

@eak12913 eak12913 merged commit c1d8f78 into master Jan 5, 2022
@eak12913 eak12913 deleted the configure-site-url-for-docusuarus branch January 5, 2022 20:17
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.

3 participants