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

feat: edit folder structure #29

Merged
merged 17 commits into from
Sep 6, 2024
Merged

feat: edit folder structure #29

merged 17 commits into from
Sep 6, 2024

Conversation

julia-rabello
Copy link
Collaborator

@julia-rabello julia-rabello commented Sep 3, 2024

EDU-13268

  • added category and subcategory folders to tutorials.
  • added product team folders to faqs and known issues.
  • added track slug folders to track articles.
  • moved the locale to the first level of folders (docs/{locale}). I also tested the other option of moving the locale to the last level of folders (.../{locale}/slug.md), but it ended up creating filepaths that were way too large for github to process on commits ( structured like ./docs/tutorials/{category}{subcategory}/{filename}/{locale}/{filename}.md), this resulted in errors when commiting.
  • retrieved docs from contentful following this new structure and applied scripts (callouts, quotes) and manual fixes (mismatched quotes, image urls).

⚠️ Content is currently not working when using the preview of this branch (https://leafy-mooncake-7c2e5e.netlify.app/api/preview?branch=edit-folder-structure). If we do decide on this folder structure, we need to make sure devportal code supports it.

Copy link
Contributor

@PedroAntunesCosta PedroAntunesCosta left a comment

Choose a reason for hiding this comment

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

LGTM! But this merge requires this PR to be merged in the helpcenter repo for the portal to work.

Copy link
Contributor

@PedroAntunesCosta PedroAntunesCosta left a comment

Choose a reason for hiding this comment

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

LGTM

@julia-rabello julia-rabello merged commit 51a2a36 into main Sep 6, 2024
1 check failed
@julia-rabello julia-rabello deleted the edit-folder-structure branch September 6, 2024 15:56
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