Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix warning on Courses CPT "Level" column if the level found doesn't have a name (probably deleted) #62

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

kimcoleman
Copy link
Member

All Submissions:

Changes proposed in this Pull Request:

There was a warning shown for levels without a "name" (probably because they were deleted) on the All Courses CPT screen in the WP Admin. This fix checks that there is an existing level name, then adds to the array.

Other information:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you successfully run tests with your changes locally?

Changelog entry

  • BUG FIX: Fixed warning in Levels column of the All Courses screen in WP admin.

Enter a summary of all changes on this Pull Request. This will appear in the changelog if accepted.

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.

1 participant