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

Move article to archived #1157

Merged
merged 9 commits into from
Aug 4, 2023
Merged

Move article to archived #1157

merged 9 commits into from
Aug 4, 2023

Conversation

ahoneybun
Copy link
Member

Addresses feedback on issue 1150 since the article is not getting update and the information is in the blog now.

@jacobgkau jacobgkau linked an issue Aug 2, 2023 that may be closed by this pull request
@jacobgkau
Copy link
Member

The build is failing with this:

Error: Error generating route "/articles/roadmap": Article not found

Just to be clear, moving it to the "archived" folder should mean it's not being generated at all, right? (As opposed to just setting to "hidden," which would mean it's still available from the search box?)

@ahoneybun
Copy link
Member Author

The build is failing with this:

Error: Error generating route "/articles/roadmap": Article not found

Just to be clear, moving it to the "archived" folder should mean it's not being generated at all, right? (As opposed to just setting to "hidden," which would mean it's still available from the search box?)

Correct, the article would appear in the search box even if it is disabled though will not appear on the page itself.

@burden
Copy link
Member

burden commented Aug 3, 2023

@ahoneybun quick heads up. Still troubleshooting, but I noticed there are links to this page in a couple spots at: content/difference-between-pop-ubuntu.md

@jacobgkau jacobgkau requested a review from a team August 3, 2023 22:42
Copy link
Member

@jacobgkau jacobgkau left a comment

Choose a reason for hiding this comment

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

This completely removes the outdated article. We can always move it back if we decide we want to start using it again.

@jacobgkau jacobgkau requested a review from a team August 3, 2023 22:43
Copy link
Contributor

@thomas-zimmerman thomas-zimmerman left a comment

Choose a reason for hiding this comment

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

looks good.

@thomas-zimmerman thomas-zimmerman merged commit 51ec690 into master Aug 4, 2023
@thomas-zimmerman thomas-zimmerman deleted the issue-1150 branch August 4, 2023 14:41
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.

System76 Roadmap is now 2 years out of date
4 participants