Skip to content

Conversation

axelbjo
Copy link
Contributor

@axelbjo axelbjo commented Oct 10, 2025

📌 Summary

Fixes https://kth-se.atlassian.net/browse/KP-449


🔍 Changes

  • Moved condition that prevented non mandatory course info from showing

✅ Checklist (Author)

  • Code builds locally without errors
  • Tests added/updated and passing
  • Linting/formatting applied
  • No sensitive data in the diff
  • No stray console.logs in the diff
  • No stray comments in the diff
  • Docs/README/CHANGELOG updated (if needed)
  • Sufficient logging
  • Errors are handled accordingly

🧪 Testing & Verification

Screenshots / recordings (if UI change):


⚠️ Impact / Risks


🚧 Out of Scope


📦 Downstream apps

@axelbjo axelbjo force-pushed the issues/KP-449-show-info-for-optional-courses branch from ecf7930 to 0616e17 Compare October 10, 2025 09:06
Copy link
Contributor

@allazis allazis left a comment

Choose a reason for hiding this comment

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

Tested locally and things looked good for Masterprogram, järnvägsteknik (TJVTM), Utbildningsplan kull HT2025, Årskurs 2 | KTH

With this example I can't see any difference from kth.se though: Masterprogram, inbyggda system (TEBSM), Utbildningsplan kull HT2026, Årskurs 1 | KTH What has been corrected there?

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.

2 participants