Skip to content

Remove pt-PT locale (only pt-BR exists)#620

Merged
adamint merged 1 commit intomainfrom
remove-pt-pt-locale
Mar 26, 2026
Merged

Remove pt-PT locale (only pt-BR exists)#620
adamint merged 1 commit intomainfrom
remove-pt-pt-locale

Conversation

@adamint
Copy link
Member

@adamint adamint commented Mar 26, 2026

Summary

Removes the pt-PT (Portuguese - Portugal) locale from the site. Only pt-BR (Portuguese - Brazil) has active translations — pt-PT was added optimistically in the initial release but never gained content beyond a single translated index.mdx. Every sidebar restructure has been copy-pasting 'pt-PT' entries by inertia.

Changes

  • locales.ts — removed pt-PT locale entry
  • lunaria.config.json — removed pt-PT tracking entry
  • astro.config.mjs — removed back-to-top label and LLMs exclude glob
  • cookie.config.ts — removed consent/preferences translations
  • All 6 sidebar topic files — removed 'pt-PT' label entries
  • src/content/docs/pt-pt/ — deleted content directory
  • translation-guide.mdx — removed pt-PT row from locale table

Copilot AI review requested due to automatic review settings March 26, 2026 22:28
Remove all pt-PT (Portuguese Portugal) references from the site:
- locales.ts locale entry
- lunaria.config.json tracking entry
- astro.config.mjs back-to-top label and LLMs exclude entry
- cookie.config.ts consent/preferences translations
- All sidebar topic files (community, dashboard, deployment, docs, integrations, reference)
- pt-pt content directory (index.mdx)
- translation-guide.mdx table row

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the unused pt-PT locale from the Aspire docs site and cleans up related configuration/sidebar translation entries. It also updates “deploy your first app” documentation by introducing dedicated C# and TypeScript AppHost deployment tutorials and rewiring several links/navigation entries to point at the new pages.

Changes:

  • Removed pt-PT locale/config entries and deleted the src/content/docs/pt-pt/ content.
  • Updated sidebar topic translation maps to drop pt-PT and adjusted the “Deploy your first app” sidebar section to include C#/TypeScript sub-pages.
  • Added new deployment tutorials (deploy-first-app-csharp, deploy-first-app-typescript) and updated multiple docs links to target them.

Reviewed changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/frontend/src/content/docs/whats-new/aspire-13.mdx Updates Python template “deploy” link to the new C# AppHost deployment tutorial.
src/frontend/src/content/docs/pt-pt/index.mdx Deletes the only remaining pt-PT docs page.
src/frontend/src/content/docs/ja/whats-new/aspire-13.mdx Updates Japanese “deploy” link to the new C# AppHost deployment tutorial.
src/frontend/src/content/docs/get-started/first-app-typescript-apphost.mdx Updates the post-quickstart CTA to point to the new TypeScript deployment tutorial.
src/frontend/src/content/docs/get-started/deploy-first-app-typescript.mdx Adds a new deployment tutorial for TypeScript AppHost.
src/frontend/src/content/docs/get-started/deploy-first-app-csharp.mdx Adds a new deployment tutorial for C# AppHost (with a C#/Python pivot).
src/frontend/src/content/docs/get-started/add-aspire-existing-app-typescript-apphost.mdx Updates deployment links to the new TypeScript deployment tutorial.
src/frontend/src/content/docs/get-started/add-aspire-existing-app-csharp-apphost.mdx Updates deployment links to the new C# deployment tutorial.
src/frontend/src/content/docs/dashboard/standalone-for-python.mdx Updates “Next steps” deployment link to the new C# deployment tutorial (Python pivot via query param).
src/frontend/src/content/docs/community/translation-guide.mdx Removes pt-pt from the supported locales table.
src/frontend/lunaria.config.json Removes pt-pt from Lunaria locale tracking.
src/frontend/config/sidebar/reference.topics.ts Removes pt-PT translation labels from the Reference sidebar topic config.
src/frontend/config/sidebar/integrations.topics.ts Removes pt-PT translation labels from the Integrations sidebar topic config.
src/frontend/config/sidebar/docs.topics.ts Removes pt-PT translations and restructures “Deploy your first app” to list overview/C#/TypeScript pages.
src/frontend/config/sidebar/deployment.topics.ts Removes pt-PT translation labels from the Deployment sidebar topic config.
src/frontend/config/sidebar/dashboard.topics.ts Removes pt-PT translation labels from the Dashboard sidebar topic config.
src/frontend/config/sidebar/community.topics.ts Removes pt-PT translation labels from the Community sidebar topic config.
src/frontend/config/locales.ts Removes the pt-pt locale entry from site locales.
src/frontend/config/cookie.config.ts Removes pt-pt cookie consent translations.
src/frontend/astro.config.mjs Removes pt-pt back-to-top label and excludes glob entry.
Comments suppressed due to low confidence (1)

src/frontend/config/sidebar/docs.topics.ts:323

  • The PR title/description focus on removing the pt-PT locale, but this change also restructures the “Deploy your first app” sidebar group and adds new C#/TypeScript deployment tutorial pages/links. Please either update the PR description/title to reflect these additional changes or split the deployment tutorial work into a separate PR to keep the locale removal focused.
          label: 'Deploy your first app',
          collapsed: false,
          translations: {
            da: 'Udrul din første app',
            de: 'Bereitstellung Ihrer ersten App',

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@adamint adamint enabled auto-merge (squash) March 26, 2026 22:37
@adamint adamint merged commit 55500e6 into main Mar 26, 2026
5 checks passed
@adamint adamint deleted the remove-pt-pt-locale branch March 26, 2026 22:44
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