Skip to content

[Feat] support for pack quantity#7

Open
Nova-Aegis wants to merge 10 commits intoalias-asso:mainfrom
Nova-Aegis:main
Open

[Feat] support for pack quantity#7
Nova-Aegis wants to merge 10 commits intoalias-asso:mainfrom
Nova-Aegis:main

Conversation

@Nova-Aegis
Copy link

@Nova-Aegis Nova-Aegis commented Oct 23, 2025

Objective

Adds support to show lowest number of remaining Courses in a pack (and the lowest total).

Preview

Dark Light

Changelog

  • Shows the lowest remaining quantity of a packs Courses.
  • Colors pack courses when none remaining
  • On course quantity modification, updates the course-grid instead of single course.
  • Reworked Pack structure to list CourseID with quantity and total per course.

@Nova-Aegis Nova-Aegis changed the title Support for pack quantity [Feat] support for pack quantity Oct 23, 2025
@Nova-Aegis Nova-Aegis marked this pull request as draft October 23, 2025 08:59
@Nova-Aegis Nova-Aegis changed the title [Feat] support for pack quantity [Feat] support for pack quantity & pack course hover tooltip Oct 26, 2025
@Nova-Aegis Nova-Aegis changed the title [Feat] support for pack quantity & pack course hover tooltip [Feat] support for pack quantity Oct 28, 2025
@Nova-Aegis Nova-Aegis marked this pull request as ready for review October 28, 2025 09:53
Copy link
Contributor

@Yopaman Yopaman left a comment

Choose a reason for hiding this comment

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

small typos, but the rest looks good

Semester *string
}

type CourseAmmount struct {
Copy link
Contributor

Choose a reason for hiding this comment

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

type ("amount" instead of ammount)

return fmt.Sprintf("%s %s %d", c.Code, c.Kind, c.Part)
}

func (pack Pack) PCPT() CourseAmmount {
Copy link
Contributor

Choose a reason for hiding this comment

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

same typo ("ammount")

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.

3 participants