Skip to content

The full Alfa java course #1892

Open
Ara225 wants to merge 42 commits into
mainfrom
alfa-java-course-the-rest
Open

The full Alfa java course #1892
Ara225 wants to merge 42 commits into
mainfrom
alfa-java-course-the-rest

Conversation

@Ara225
Copy link
Copy Markdown
Contributor

@Ara225 Ara225 commented May 25, 2026

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
Copy link
Copy Markdown

netlify Bot commented May 25, 2026

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
Copy link
Copy Markdown

netlify Bot commented May 25, 2026

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
Copy link
Copy Markdown
Contributor Author

Ara225 commented May 27, 2026

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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 📋 Backlog

Development

Successfully merging this pull request may close these issues.

2 participants