Skip to content

Conversation

@DanBloxham-sw
Copy link
Contributor

The different Count values in the course statistics were previously not restricted to candidates the specified centre. Now, the candidates should only count towards the Count values when the record's CentreID matches. This should correct some courses with the "AllCentres" flag from appearing to have more delegates taking the course then are at the centre.

Copy link
Contributor

@JBsoftwire JBsoftwire left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@stellake stellake left a comment

Choose a reason for hiding this comment

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

Looks good! 👍

@DanBloxham-sw DanBloxham-sw merged commit e22830b into master Jul 14, 2021
@DanBloxham-sw DanBloxham-sw deleted the HEEDLS-470-TopCoursesCandidateCountBugFix branch July 14, 2021 12:07
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