Skip to content

Conversation

@akien-mga
Copy link
Member

Draft for now, as we should discuss what to do with the "Compiling for UWP" and "Exporting for UWP" pages.

They were already stubs clarifying that this is not supported yet. We may want to keep them to clarify that the platform support was fully dropped for 4.x, and avoid having 404 errors.
At the same time, how long do we want to keep mentions of removed platform support in the docs?

Shouldn't be cherry-picked.

@akien-mga akien-mga added enhancement content:organization Issues and PRs related to reorganizing the content area:manual Issues and PRs related to the Manual/Tutorials section of the documentation area:engine details Issues and PRs related to the Engine Details section of the documentation labels Sep 7, 2023
@akien-mga akien-mga added this to the 4.2 milestone Sep 7, 2023
@mhilbrunner
Copy link
Member

mhilbrunner commented Sep 11, 2023

At the same time, how long do we want to keep mentions of removed platform support in the docs?

If it is on a single "Supported platforms" section, I'm undecided whether it hurts to just list discontinued platforms with the versions where we first dropped support or that last supported them kinda "forever".

Theoretically, we could redirect the "Compiling for UWP" and "Exporting for UWP" pages there then as an explanation.
I'm also fine with keeping them around as stubs for now just saying UWP was dropped, but eventually we should maybe just remove those, seeing as UWP was never supported in 4.x in the first place.

Copy link
Member

@mhilbrunner mhilbrunner left a comment

Choose a reason for hiding this comment

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

Besides the above discussion the changes look good to me once we come to a decision. :)

@akien-mga
Copy link
Member Author

Added a note in the FAQ about the UWP removal.

We can maybe add redirects that point to that section of the FAQ? Can you handle this @mhilbrunner?

@akien-mga akien-mga marked this pull request as ready for review November 16, 2023 12:01
@mhilbrunner mhilbrunner merged commit 7f97b67 into godotengine:master Nov 18, 2023
@mhilbrunner mhilbrunner added the content:redirect Issues and PRs involving moving content and adding a redirect rule on the backend label Nov 18, 2023
@mhilbrunner
Copy link
Member

Will do. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:engine details Issues and PRs related to the Engine Details section of the documentation area:manual Issues and PRs related to the Manual/Tutorials section of the documentation content:organization Issues and PRs related to reorganizing the content content:redirect Issues and PRs involving moving content and adding a redirect rule on the backend enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants