Skip to content

The full Alfa java course #1892

Merged
illicitonion merged 42 commits into
mainfrom
alfa-java-course-the-rest
Jun 10, 2026
Merged

The full Alfa java course #1892
illicitonion merged 42 commits into
mainfrom
alfa-java-course-the-rest

Conversation

@Ara225

@Ara225 Ara225 commented May 25, 2026

Copy link
Copy Markdown
Contributor

This builds and I can't see anything obviously wrong apart from the fact that backlog issues aren't showing up on my local ( I added the relevant label) and this this warning that doesn't make a lot of sense:

WARN "/home/anna/git_linux/curriculm_NOT_A_FORK/org-cyf/content/tracks/java/success/index.md:1:1": duplicate menu entry with identifier "End of Module Review" in menu "success criteria"

And there is the TODO they left in topic 4:  Alfa Streams and Lambdas exercises - TODO when we move this to git: need the superclass and the imports, as well as the source for NameFinder, ShapeCounter, Remainders..

@netlify

netlify Bot commented May 25, 2026

Copy link
Copy Markdown

Deploy Preview for cyf-common ready!

Name Link
🔨 Latest commit ec051c0
🔍 Latest deploy log https://app.netlify.com/projects/cyf-common/deploys/6a172342878b12000846b51f
😎 Deploy Preview https://deploy-preview-1892--cyf-common.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@netlify

netlify Bot commented May 25, 2026

Copy link
Copy Markdown

Deploy Preview for cyf-curriculum ready!

Name Link
🔨 Latest commit ec051c0
🔍 Latest deploy log https://app.netlify.com/projects/cyf-curriculum/deploys/6a1723425a4ff000082aae6b
😎 Deploy Preview https://deploy-preview-1892--cyf-curriculum.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 85 (🔴 down 12 from production)
Accessibility: 100 (no change from production)
Best Practices: 92 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports
🤖 Make changes Run an agent on this branch

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

@Ara225 Ara225 requested a review from illicitonion May 25, 2026 22:49
@illicitonion

Copy link
Copy Markdown
Member

I've skimmed this, not gone word-by-word, but it looks great! I pushed a few small commits, I hope you don't mind, feel free to review them and complain!

A range of thoughts:

  • We probably won't follow the day-plans precisely, that's ok, we'll see how they go and evolve them
  • We've ended up with extra sprints... I see how this happened - in the doc we had a sprint 2b... I will ask the Alfa folks about this :D
  • Sprint 2b/3's backlog seems missing? I guess this goes back to the sprint 2b vs 3 question?
  • Sprint 8 is missing a backlog? I'm assuming that's missing from the source material from Alfa too?

@Ara225

Ara225 commented May 27, 2026

Copy link
Copy Markdown
Contributor Author

Hi,
Thanks for that, good catches

Yeah the day plans and the timing for the prep items probably aren't that great but the source is a bit thin on details in that bit - probably best worked out in practice anyway

There doesn’t seem to be a sprint 2b in the word doc for me, just 1-8, could we have diffrent versions?

On sprint 3 they've just written a sentence saying to submit excercises as the coursework, so I created a stub ticket saying that, wasn't 100% sure how to handle the line bluring and that seemed the least fuss.

Sprint 8 stops where excercises should be for the last topic so there's no backlog present

@illicitonion

Copy link
Copy Markdown
Member

All makes sense - I was indeed looking at an older draft

For the sprint 3 exercises - can you file a ticket like CodeYourFuture/Module-Tools#178 which can act as a placeholder?

For sprint 8, let's worry about this after we merge!

I'll pull in the Alfa folks to take a closer look, then let's get this thing going!

@Ara225

Ara225 commented May 31, 2026

Copy link
Copy Markdown
Contributor Author

@illicitonion Sorry for the delay in getting back to you. Sounds good, edited the stub issue I created to be in that format CodeYourFuture/Module-Java#5 (comment)

@illicitonion illicitonion left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Let's merge this and get it deployed - we can make any edits we need in the future :) Thanks again so much for putting this together!

@github-project-automation github-project-automation Bot moved this from 📋 Backlog to 👀 In review in Curriculum Planning Jun 10, 2026
@illicitonion illicitonion merged commit 7232662 into main Jun 10, 2026
11 checks passed
@illicitonion illicitonion deleted the alfa-java-course-the-rest branch June 10, 2026 15:30
@github-project-automation github-project-automation Bot moved this from 👀 In review to ✅ Done in Curriculum Planning Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants