Skip to content

Conversation

@nul800sebastiaan
Copy link
Member

Prerequisites

  • I have added steps to test this contribution in the description below

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 (in NewsDashboardService). It should now look like this:

image

In the Settings section, we also moved to the Our forum:

image

And when doing an upgrade, the compare link ("View report) now also links to the new releases pages instead of to Our:

image

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.cshtml and NotFound.cshtml have 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.

Copilot AI review requested due to automatic review settings October 24, 2025 12:35
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 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 HelpController and 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'.

Copy link
Contributor

@AndyButland AndyButland left a 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.

@nul800sebastiaan
Copy link
Member Author

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?

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!

@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.

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!

@nul800sebastiaan nul800sebastiaan added release/no-notes Not directly part of the release (updating README, build scripts, tests, etc.) type/improvement release/17.0.0 labels Oct 25, 2025
@nul800sebastiaan
Copy link
Member Author

nul800sebastiaan commented Oct 25, 2025

Now updated it, replacing the double forum mention to a link to the community site. Also made the headings in the boxes a bit bigger and updated the descriptions to say something.. more descriptive 😉

Again, these only show up when the remote dashboard can't be loaded, so you need to break the link in /src/Umbraco.Web.UI.Client/src/packages/core/backend-api/sdk.gen.ts (NewsDashboardService) to test it out, or disconnect your internet. Anyway, you will see the following:

image

Copy link
Contributor

@AndyButland AndyButland left a 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.

@iOvergaard
Copy link
Contributor

@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?

@nul800sebastiaan
Copy link
Member Author

@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.

@AndyButland AndyButland enabled auto-merge (squash) October 27, 2025 11:01
@AndyButland AndyButland merged commit 399431b into release/17.0 Oct 27, 2025
23 checks passed
@AndyButland AndyButland deleted the task/our-to-forum branch October 27, 2025 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release/no-notes Not directly part of the release (updating README, build scripts, tests, etc.) release/17.0.0 type/improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants