Skip to content

Conversation

mansona
Copy link
Member

@mansona mansona commented May 30, 2025

The @ember-learn/tooling-core-team is in the process of migrating the guides to GJS 🎉 this is the tracking branch that can be used as a target for smaller PRs that convert individual parts of the guides to GJS

Warning

This migration assumes that the generators for GJS are in place in both ember-source and ember-cli. The core teams are working to get the generators ready for the upcoming 6.5 release so this PR should not be merged before Ember 6.5 is released.

Copy link

netlify bot commented May 30, 2025

Deploy Preview for ember-guides ready!

Name Link
🔨 Latest commit 90de3a1
🔍 Latest deploy log https://app.netlify.com/projects/ember-guides/deploys/688d1bd95b713c00082dd4e6
😎 Deploy Preview https://deploy-preview-2117--ember-guides.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

locks and others added 8 commits June 20, 2025 16:33
* Updated index to use gjs
* Updated introduction to use gjs
* Updated arguments and attributes to use gjs
* Updated conditional content to use gjs
* Updated block content to use gjs
* Updated helper functions to use gjs
* Updated component state and actions to use gjs
* Updated looping to use gjs
* Updated built in components to use gjs

Removed ai helper
* fixed whitespace
* added missing imports
* fixed import paths
@tcjr
Copy link
Contributor

tcjr commented Jul 16, 2025

For an idea of the level of effort here, these are counts of handlebars code blocks in the guides/release/* markdown files. This might not be everything as some components might be just JavaScript. But it's a rough guideline.

Total: 274 code blocks in 42 files.

Here is the status of getting these to gjs (num-codeblocks - filename):

All done 🎉

Already Merged

PR #2124

  • 13 - components/block-content.md
  • 15 - components/built-in-components.md
  • 12 - components/component-arguments-and-html-attributes.md
  • 12 - components/component-state-and-actions.md
  • 13 - components/conditional-content.md
  • 15 - components/helper-functions.md
  • 17 - components/introducing-components.md
  • 11 - components/looping-through-lists.md
  • 13 - components/template-lifecycle-dom-and-modifiers.md
  • 2 - components/template-tag-format.md

PR #2126

  • 2 - accessibility/page-template-considerations.md
  • 1 - services/index.md
  • 1 - typescript/additional-resources/gotchas.md
  • 1 - typescript/application-development/testing.md
  • 4 - typescript/core-concepts/invokables.md
  • 1 - models/creating-updating-and-deleting-records.md N/A
  • 1 - models/defining-models.md N/A
  • 2 - models/index.md
  • 2 - models/relationships.md N/A

PR #2127

  • 4 - in-depth-topics/autotracking-in-depth.md
  • 8 - in-depth-topics/patterns-for-actions.md
  • 11 - in-depth-topics/patterns-for-components.md
  • 11 - in-depth-topics/rendering-values.md

PR #2118

  • 3 - routing/controllers.md
  • 8 - routing/defining-your-routes.md
  • 5 - routing/linking-between-routes.md
  • 3 - routing/query-params.md
  • 4 - routing/specifying-a-routes-model.md

This repo

Tutorial - generated from super rentals

  • 7 - tutorial/part-1/building-pages.md
  • 13 - tutorial/part-1/component-basics.md
  • 8 - tutorial/part-1/interactive-components.md
  • 5 - tutorial/part-1/more-about-components.md
  • 7 - tutorial/part-1/orientation.md
  • 4 - tutorial/part-1/reusable-components.md
  • 7 - tutorial/part-1/working-with-data.md
  • 4 - tutorial/part-2/provider-components.md
  • 3 - tutorial/part-2/route-params.md
  • 2 - tutorial/part-2/service-injection.md

Copy link

netlify bot commented Sep 12, 2025

Deploy Preview for ember-guides ready!

Name Link
🔨 Latest commit e59d0f5
🔍 Latest deploy log https://app.netlify.com/projects/ember-guides/deploys/68d7b53c3dc1270008903d81
😎 Deploy Preview https://deploy-preview-2117--ember-guides.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

mansona and others added 18 commits September 13, 2025 11:20
update anatomy-of-an-ember-app to gjs
Co-authored-by: Katie Gengler <[email protected]>
* Updated chapter 1 to strict

* Updated chapter 2 to strict

* Updated chapter 3 to strict

* Partially updated chapter 4 to strict

* Partially updated chapter 4 to strict

* Partially updated chapter 4 to strict

* Partially updated chapter 5 to strict

* WIP

* Partially updated chapter 5 to strict

* Fixed bad patch

* Partially updated chapter 6 to strict

* Partially updated chapter 6 to strict

* Partially updated chapter 6 to strict

* Partially updated chapter 7 to strict

* Partially updated chapter 7 to strict

* Updated chapter 8 to strict

* Updated chapter 9 to strict

* Updated chapter 10 to strict

* Updated chapter 12 to strict

---

Commit:  ember-learn/super-rentals-tutorial@cba238c
Script:  https://github.com/ember-learn/super-rentals-tutorial/blob/cba238c129c150a3bc0b63dfe8828ed7d3a33ab1/.github/workflows/build.yml
Logs:    https://github.com/ember-learn/super-rentals-tutorial/commit/cba238c129c150a3bc0b63dfe8828ed7d3a33ab1/checks

# Conflicts:
#	guides/release/tutorial/part-1/component-basics.md
#	guides/release/tutorial/part-1/orientation.md
Apply the changes to the tutorial for the gjs branch
Copy link
Member

@jaredgalanis jaredgalanis left a comment

Choose a reason for hiding this comment

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

Did a relatively surface level scan and it all looks okay to me. Approving so that we can move forward with this and perform any fixes needed in follow up.

@NullVoxPopuli
Copy link
Contributor

this PR is too big for GH to render 🙈 let's merge and fix forward, probably?

@mansona mansona marked this pull request as ready for review September 27, 2025 09:58
@mansona
Copy link
Member Author

mansona commented Sep 27, 2025

We decided at the learning team meeting yesterday that we are merging this PR and any fixes can be done on a fix-forward basis.

Thank you everyone for your work in getting this over the line 🎉

@mansona mansona merged commit 4ad282a into master Sep 27, 2025
6 of 8 checks passed
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.

7 participants