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

Redirect pathway to MDN Curriculum and remove links to it #32513

Merged
merged 3 commits into from
Mar 5, 2024

Conversation

chrisdavidmills
Copy link
Contributor

@chrisdavidmills chrisdavidmills commented Mar 1, 2024

Description

As part of launching the MDN curriculum, we are making some changes to the main MDN content, as discussed in https://github.com/orgs/mdn/discussions/633 (see the "Changes we intend to make" heading).

This PR implements points 1 and 3 from that list — removing the front-end web developer pathway from MDN, redirecting its URL to the MDN curriculum (which supercedes it), removing all the note boxes that pointed to it from the site, and adding a link to the MDN Curriculum to the front page only.

Motivation

Additional details

Related issues and pull requests

@chrisdavidmills chrisdavidmills requested review from a team as code owners March 1, 2024 17:12
@chrisdavidmills chrisdavidmills requested review from Josh-Cena, estelle, hamishwillee and fiji-flo and removed request for a team and Josh-Cena March 1, 2024 17:12
@github-actions github-actions bot added Content:CSS Cascading Style Sheets docs Content:HTML Hypertext Markup Language docs Content:JS JavaScript docs Content:Learn Learning area docs Content:Learn:Client-side Content under “Client-side JavaScript frameworks” (Svelte, React, Angular, Vue) and related subtrees Content:Learn:JavaScript Learning area JavaScript docs Content:Learn:CSS Learning area CSS docs Content:Learn:HTML Learning area HTML docs Content:Learn:Accessibility Accessibility documentation in the Learn Area Content:Learn:Forms Learning area Forms docs Content:Learn:Cross-Browser-Testing Learning area Cross-Browser-Testing docs Content:Learn:GitHub size/m [PR only] 51-500 LoC changed labels Mar 1, 2024
@chrisdavidmills
Copy link
Contributor Author

@fiji-flo , adding you here so that you can check whether I've done the redirect correctly!

Copy link
Contributor

github-actions bot commented Mar 1, 2024

Preview URLs (28 pages)
Flaws (2)

Note! 27 documents with no flaws that don't need to be listed. 🎉

URL: /en-US/docs/Learn
Title: Learn web development
Flaw count: 2

  • broken_links:
    • Can't resolve /en-US/curriculum/
    • Can't resolve /en-US/curriculum/
External URLs (93)

URL: /en-US/docs/Learn/MathML
Title: Writing mathematics with MathML


URL: /en-US/docs/Learn/MathML/First_steps
Title: MathML first steps overview


URL: /en-US/docs/Learn/Forms
Title: Web forms — Working with user data


URL: /en-US/docs/Learn/Accessibility
Title: Accessibility


URL: /en-US/docs/Learn/JavaScript
Title: JavaScript — Dynamic client-side scripting


URL: /en-US/docs/Learn/JavaScript/Objects
Title: Introducing JavaScript objects


URL: /en-US/docs/Learn/JavaScript/Asynchronous
Title: Asynchronous JavaScript


URL: /en-US/docs/Learn/JavaScript/First_steps
Title: JavaScript First Steps


URL: /en-US/docs/Learn/JavaScript/Building_blocks
Title: JavaScript building blocks


URL: /en-US/docs/Learn/JavaScript/Client-side_web_APIs
Title: Client-side web APIs


URL: /en-US/docs/Learn/HTML/Tables
Title: HTML tables


URL: /en-US/docs/Learn/HTML/Multimedia_and_embedding
Title: Multimedia and embedding


URL: /en-US/docs/Learn/HTML/Introduction_to_HTML
Title: Introduction to HTML


URL: /en-US/docs/Learn/CSS/CSS_layout
Title: CSS layout


URL: /en-US/docs/Learn/CSS/Styling_text
Title: CSS styling text


URL: /en-US/docs/Learn/CSS/First_steps
Title: CSS first steps overview


URL: /en-US/docs/Learn/CSS/Building_blocks
Title: CSS building blocks


URL: /en-US/docs/Learn/Tools_and_testing/Client-side_JavaScript_frameworks
Title: Understanding client-side JavaScript frameworks


URL: /en-US/docs/Learn/Tools_and_testing/GitHub
Title: Git and GitHub


URL: /en-US/docs/Web/JavaScript
Title: JavaScript


URL: /en-US/docs/Web/CSS
Title: CSS: Cascading Style Sheets

(comment last updated: 2024-03-04 15:48:47)

Copy link
Contributor

@fiji-flo fiji-flo left a comment

Choose a reason for hiding this comment

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

👍

@@ -5244,6 +5244,7 @@
/en-US/docs/Learn/Forms/Your_first_HTML_form /en-US/docs/Learn/Forms/Your_first_form
/en-US/docs/Learn/Forms/Your_first_HTML_form/Example /en-US/docs/Learn/Forms/Your_first_form
/en-US/docs/Learn/Forms/Your_first_form/Example /en-US/docs/Learn/Forms/Your_first_form
/en-US/docs/Learn/Front-end_web_developer https://developer.mozilla.org/en-US/curriculum/
Copy link
Contributor

Choose a reason for hiding this comment

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

That's not ideal. But okay for now. We might want to change yari there.

@chrisdavidmills chrisdavidmills merged commit 26e2f98 into mdn:main Mar 5, 2024
10 checks passed
@chrisdavidmills chrisdavidmills deleted the remove-redirect-pathway branch March 5, 2024 14:49
yin1999 added a commit to mdn/translated-content that referenced this pull request Mar 6, 2024
yin1999 added a commit to mdn/translated-content that referenced this pull request Mar 6, 2024
* zh-cn: sync translated content

* Redirect pathway to MDN Curriculum and remove links to it

ref: mdn/content#32513

* Remove page for deprecated `<image>` element

ref: mdn/content#32553

---------

Co-authored-by: Allo <[email protected]>
ciyoe pushed a commit to ciyoe/translated-content that referenced this pull request Apr 20, 2024
* zh-cn: sync translated content

* Redirect pathway to MDN Curriculum and remove links to it

ref: mdn/content#32513

* Remove page for deprecated `<image>` element

ref: mdn/content#32553

---------

Co-authored-by: Allo <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:CSS Cascading Style Sheets docs Content:HTML Hypertext Markup Language docs Content:JS JavaScript docs Content:Learn:Accessibility Accessibility documentation in the Learn Area Content:Learn:Client-side Content under “Client-side JavaScript frameworks” (Svelte, React, Angular, Vue) and related subtrees Content:Learn:Cross-Browser-Testing Learning area Cross-Browser-Testing docs Content:Learn:CSS Learning area CSS docs Content:Learn:Forms Learning area Forms docs Content:Learn:GitHub Content:Learn:HTML Learning area HTML docs Content:Learn:JavaScript Learning area JavaScript docs Content:Learn Learning area docs size/m [PR only] 51-500 LoC changed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Info box linking to FE Web Developer course is repeated in multiple pages
3 participants