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

[develop] Update docs URLs #4294

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

alhambrav
Copy link
Member

Ticket reference or full description of what's in the PR

Update docs URLs

@sumerjabri
Copy link
Member

@jvega190 please link the new ticket to remove the old files.

@jvega190
Copy link
Member

Copy link
Member

@sumerjabri sumerjabri left a comment

Choose a reason for hiding this comment

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

The READMEs have changed, see the PR: https://github.com/craftercms/studio-ui/pull/4358/files

@@ -68,7 +68,7 @@ export function AboutCrafterCMSView() {
values={{
a: (msg) => (
<Link
href={`https://docs.craftercms.org/en/${env.packageVersion?.substr(
href={`https://craftercms.com/docs/${env.packageVersion?.substr(
Copy link
Member

Choose a reason for hiding this comment

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

This won't work. Docs now are /docs/{major}.

This must also take into account a major version of more than 1 char, for example version 10.

@@ -85,7 +85,7 @@ function LauncherGlobalNav(props: LauncherGlobalNavProps) {
<LauncherTile
title={formatMessage(messages.docs)}
icon={{ id: 'craftercms.icons.Docs' }}
link={`https://docs.craftercms.org/en/${getSimplifiedVersion(version)}/index.html`}
link={`https://craftercms.com/docs/${getSimplifiedVersion(version)}/index.html`}
Copy link
Member

Choose a reason for hiding this comment

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

This also won't work. See the new method Roy added on 2/20/2025.

@@ -212,7 +212,7 @@ function QuickCreateSection(props: QuickCreateSectionProps) {
{version && (
<Button
size="small"
href={`https://docs.craftercms.org/en/${version}/developers/content-modeling.html#setting-up-quick-create`}
href={`https://craftercms.com/docs/${version}/by-role/developer/common/content-modeling/content-modeling.html#quick-create`}
Copy link
Member

Choose a reason for hiding this comment

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

This won't work either.

@@ -533,17 +533,17 @@ export const dragAndDropMessages = defineMessages({
contentTypeNotFound: {
id: 'dragAndDropMessages.contentTypeNotFound',
defaultMessage:
'The content type id was not found in your template. Drag and drop will be impaired. More info at docs.craftercms.org/en/3.1/system-administrators/upgrade/index.html.'
'The content type id was not found in your template. Drag and drop will be impaired. More info at craftercms.com/docs/3.1/system-administrators/upgrade/index.html.'
Copy link
Member

Choose a reason for hiding this comment

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

3.1?

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.

4 participants