Skip to content

Only return published course associations#1519

Merged
thomasplevy merged 6 commits into
devfrom
autoenroll-drafts
Feb 8, 2021
Merged

Only return published course associations#1519
thomasplevy merged 6 commits into
devfrom
autoenroll-drafts

Conversation

@thomasplevy

Copy link
Copy Markdown
Contributor

Description

Fixes https://github.com/gocodebox/lifterlms-groups/issues/135

How has this been tested?

New unit tests
Manual inspection of related areas to avoid feature regressions

Types of changes

Bug fix

Checklist:

  • My code has been tested.
  • My code passes all existing automated tests.
  • My code follows the LifterLMS Coding & Documentation Standards.

@thomasplevy thomasplevy self-assigned this Feb 5, 2021

@eri-trabiccolo eri-trabiccolo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

docs, minor

Comment thread includes/models/model.llms.membership.php Outdated
Comment thread includes/models/model.llms.membership.php Outdated
Comment thread tests/phpunit/unit-tests/models/class-llms-test-model-llms-membership.php Outdated
Thomas Patrick Levy and others added 3 commits February 8, 2021 10:56
@thomasplevy thomasplevy merged commit 8fea4d0 into dev Feb 8, 2021
@thomasplevy thomasplevy deleted the autoenroll-drafts branch February 8, 2021 20:13
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