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

Invalid prop warning in Learner > Library > Category #8810

Closed
sairina opened this issue Dec 6, 2021 · 1 comment · Fixed by #8819
Closed

Invalid prop warning in Learner > Library > Category #8810

sairina opened this issue Dec 6, 2021 · 1 comment · Fixed by #8819
Assignees
Labels
TAG: dev experience Build performance, linting, debugging...
Milestone

Comments

@sairina
Copy link
Contributor

sairina commented Dec 6, 2021

Observed behavior

When in Learner > Library, and I click on "Categories" in the side panel, there is a warning in the console that pops up for anything that is not "All categories" or "None of the above":
Invalid prop: type check failed for prop "span". Expected String with value "4", got Number with value 4. for

Errors and logs

Screen Shot 2021-12-06 at 12 33 13 PM

Expected behavior

No errors would show up in console.

User-facing consequences

Cluttered console

Steps to reproduce

  1. Sign in as learner
  2. Click on Library
  3. Open the console in browser
  4. Under "Category," click on anything other than "All categories" or "None of the above"

Context

  • Kolibri v0.15.x
  • Chrome
@sairina sairina added the TAG: dev experience Build performance, linting, debugging... label Dec 6, 2021
@sairina sairina added this to the 0.15.0 milestone Dec 6, 2021
@rtibbles
Copy link
Member

rtibbles commented Dec 6, 2021

Note that this is likely :span="4" being passed instead of span="4" to a KGridItem component.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TAG: dev experience Build performance, linting, debugging...
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants