Skip to content

feat(lessons/12): use luxon instead of moment#15

Merged
uncenter merged 6 commits intouncenter:devfrom
SteelMaverick:Lesson12Luxon
Dec 29, 2025
Merged

feat(lessons/12): use luxon instead of moment#15
uncenter merged 6 commits intouncenter:devfrom
SteelMaverick:Lesson12Luxon

Conversation

@SteelMaverick
Copy link
Contributor

This contains the library update from Moment to Luxon. Just a heads up however that the code snippets provided have also been updated to use ESM over CJS for both the filters and Eleventy configuration file. I have altered some of the content to briefly describe how Luxon works as well as the OrdinalDay() function. I also included an info box briefly talking about the past implementation of Moment and why the change to Luxon was made.

@uncenter
Copy link
Owner

uncenter commented Feb 5, 2025

Please keep the JavaScript snippets in CJS for now. I'll do everything at once to ESM for #7, but having one page be ESM and the rest CJS won't be a great experience.

@SteelMaverick
Copy link
Contributor Author

I am unsure how the progress going for that is but if you want, I can make the updates up to Lesson17 (pretty much just HTML component as that was as far as I got) for ESM if you haven't gotten around to that yet and could potentially keep this open until then. Otherwise I'll get around to converting this back to CJS later today.

@uncenter
Copy link
Owner

uncenter commented Feb 5, 2025

I am unsure how the progress going for that is but if you want

Not much unfortunately, I don't have all that much time to spend on this project these days besides reviewing contributions like this one! I would love if you could update the rest of them - you can either leave this PR open and raise a new one converting every other lesson to CJS besides lesson 12, then merge both at the same-ish time, or you could convert this to CJS, I can merge soon, and then the full ESM rewrite could be merged later down the road. Up to you.

@nehimia
Copy link

nehimia commented Jul 5, 2025

Can you add me in your project?

@uncenter
Copy link
Owner

uncenter commented Jul 7, 2025

Can you add me in your project?

What does this mean?

@uncenter uncenter changed the base branch from main to dev July 7, 2025 01:32
@uncenter uncenter linked an issue Jul 7, 2025 that may be closed by this pull request
@uncenter uncenter changed the title Updated Lesson12 to use Luxon over Moment feat(lessons/12): use luxon instead of moment Jul 7, 2025
@uncenter uncenter mentioned this pull request Jul 27, 2025
1 task
@uncenter
Copy link
Owner

uncenter commented Dec 29, 2025

Thanks so much for working on this!

@uncenter uncenter merged commit c1a6460 into uncenter:dev Dec 29, 2025
uncenter added a commit that referenced this pull request Jan 2, 2026
Co-authored-by: uncenter <uncenter@uncenter.dev>
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.

Lesson 12: Replace Moment.js with Luxon

3 participants