Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs(rss): links within RSS - add BaseURL to config.toml #1732

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

aimeeu
Copy link
Contributor

@aimeeu aimeeu commented Dec 7, 2022

add BaseURL to config.toml to see if that resolves the in the RSS feed being relative (doesn't have domain)
Resolves Jira: DOC-688

@aimeeu aimeeu requested a review from a team as a code owner December 7, 2022 19:42
@aimeeu aimeeu marked this pull request as draft December 7, 2022 19:42
@aimeeu aimeeu changed the title docs(rss): add BaseURL to config.toml docs(rss): links with RSS - add BaseURL to config.toml Dec 8, 2022
@aimeeu aimeeu changed the title docs(rss): links with RSS - add BaseURL to config.toml docs(rss): links within RSS - add BaseURL to config.toml Dec 8, 2022
@link108 link108 self-requested a review March 8, 2023 02:07
@link108
Copy link
Member

link108 commented Mar 8, 2023

I just tested the RSS feed out and it wasn't working, but that brought me to this PR 👍 I checked out the netlify preview, but I wasn't able to confirm if the RSS feed was updated - the RSS button points to docs.armory.io instead of the netlify preview. That isn't a blocker, but maybe we have to test this in prod, unless there's a way to test it out via netlify 🤔

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.

2 participants