-
Couldn't load subscription status.
- Fork 2.8k
Remove references to Our Umbraco - moved to the new forum #20644
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
Conversation
There was a problem hiding this 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 references to "Our Umbraco" throughout the codebase and replaces them with links to the new forum (forum.umbraco.com) and releases page (releases.umbraco.com). The changes reflect a migration away from the old community platform to the new forum infrastructure.
Key changes:
- Updated forum and community links from our.umbraco.com to forum.umbraco.com
- Updated release comparison URLs from Our Umbraco to releases.umbraco.com
- De-emphasized Discord in favor of directing users to the forum for Q&A
- Removed unused
HelpControllerand related API endpoints that referenced Our Umbraco - Cleaned up unused translation keys across all language files that referenced the old platform
Reviewed Changes
Copilot reviewed 36 out of 36 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| tests/Umbraco.Tests.Integration/ManagementApi/Help/GetHelpControllerTests.cs | Removed test file for deleted HelpController |
| tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/Settings/Dashboard/Welcome.spec.ts | Removed Discord community link test |
| src/Umbraco.Web.UI.Client/src/packages/umbraco-news/umbraco-news-dashboard.element.ts | Updated dashboard links and labels from "Our Umbraco" to "Umbraco Forum" |
| src/Umbraco.Web.UI.Client/src/packages/settings/welcome/settings-welcome-dashboard.element.ts | Removed Discord community button |
| src/Umbraco.Web.UI.Client/src/packages/help/menu/manifests.ts | Changed help menu item from "Community Website" to "Community Forum" with updated URL |
| src/Umbraco.Web.UI.Client/src/assets/lang/*.ts | Removed unused translation keys referencing Our Umbraco, umbraco.tv, and old help/video tutorial content across all language files |
| src/Umbraco.Cms.StaticAssets/umbraco/UmbracoWebsite/NotFound.cshtml | Updated community section to reference forum instead of Our Umbraco |
| src/Umbraco.Cms.StaticAssets/umbraco/UmbracoWebsite/NoNodes.cshtml | Updated community section to reference forum instead of Our Umbraco |
| src/Umbraco.Cms.Api.Management/ViewModels/Installer/UpgradeSettingsResponseModel.cs | Changed release comparison URL from our.umbraco.com to releases.umbraco.com |
| src/Umbraco.Cms.Api.Management/OpenApi.json | Removed Help API endpoint from OpenAPI specification |
| src/Umbraco.Cms.Api.Management/Controllers/Help/HelpControllerBase.cs | Deleted unused controller base |
| src/Umbraco.Cms.Api.Management/Controllers/Help/GetHelpController.cs | Deleted unused controller implementation |
| .github/contributing-first-issue.md | Updated forum link from our.umbraco.com to forum.umbraco.com |
| .github/BUILD.md | Updated download link from our.umbraco.com to releases.umbraco.com |
Comments suppressed due to low confidence (1)
src/Umbraco.Web.UI.Client/src/packages/help/menu/manifests.ts:1
- Corrected spelling of 'Forumn' to 'Forum'.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @nul800sebastiaan for cleaning this up, it's good to get it up to date for 17.
Apart from the small inline point, I noted that we have the forum now twice on the default boxes. Do you think something else could go in the bottom box?
@NillasKA - do you think you could check in with @nul800sebastiaan next week too please so we update the live news that comes of the marketing Cloud project, to also refresh the box that talks about our.umbraco.com? Marketing will run this moving forward, but it makes sense I think to get the default content about the community how devrel would like it. It doesn't have to be just one entry that spans 4 columns - could be 2 and 2 if that works better.
Co-authored-by: Andy Butland <[email protected]>
Ah yes, I just saw that as well, it was already doubles up apparently. We should probably point to the Community site instead there, as it will show all the other destinations into the world of Umbraco. I'll update!
I don't have the access to that site and I think we (DevRel) should work on that with Marketing before the v17 release. I'll coordinate! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Had a second look with the updates, all looks good to me.
|
@AndyButland @nul800sebastiaan > The changes to the language files are technically breaking, too. However, I cannot imagine any people actually using those sections, so it may be OK to merge them in for this new major, so we don't have to carry them around for the entirety of the LTS? |
|
@iOvergaard Since translations can be customized I would not consider these breaking changes, as they can be user-amended is they really miss these exact translation keys. Ps. I will suggest that we need a descriptive breaking change policy as I've been unable to find one. For example, I'd have no idea how to mark translation keys as deprecated if we ever wanted to remove them properly. |

Prerequisites
If there's an existing issue for this PR then this fixes
Description
To test the content dashboard (only shown when it can't connect to the remote server), break the link in
\src\Umbraco.Web.UI.Client\src\packages\core\backend-api\sdk.gen.ts(inNewsDashboardService). It should now look like this:In the Settings section, we also moved to the Our forum:
And when doing an upgrade, the compare link ("View report) now also links to the new releases pages instead of to Our:
Additionally, I found some more links in some markdown files which have now been corrected.
There was also links in the
HelpController- this one is no longer in use, it's an old one that eventually came from v7 I guess. I've removed it and it's unused reference in Swagger.NoNodes.cshtmlandNotFound.cshtmlhave been updated to point to the new forum.I found a lot of completely unused translation items that referred to Our and removed those keys and also de-emphasized Discord as we would like people to do Q&A in the forum instead.