Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Multiple warnings for "Missing required prop" #815

Closed
1 task done
krysal opened this issue Feb 4, 2022 · 0 comments · Fixed by #816
Closed
1 task done

Multiple warnings for "Missing required prop" #815

krysal opened this issue Feb 4, 2022 · 0 comments · Fixed by #816
Assignees
Labels
💻 aspect: code Concerns the software code in the repository 🛠 goal: fix Bug fix 🟨 priority: medium Not blocking but should be addressed soon

Comments

@krysal
Copy link
Member

krysal commented Feb 4, 2022

Description

  • In homepage: [Vue warn]: Missing required prop: "isSearchRoute". Nothing happens visually but the error is repeated in the browser console every time a character is entered in/removed from the input field.
  • In error layout: [Vue warn]: Missing required prop: "size". This causes 404 Searchbar styles are broken #786.

These errors complicate debugging others, so they should be fixed soon (preferably).

Reproduction

  1. Navigate to each page with the developer tools open in the 'Console' tab
  2. See the errors.

Expectation

All components should have their required properties passed or set an adequate default value.

Resolution

  • 🙋 I would be interested in resolving this bug.
@krysal krysal added 🟧 priority: high Stalls work on the project or its dependents 🛠 goal: fix Bug fix 💻 aspect: code Concerns the software code in the repository labels Feb 4, 2022
@krysal krysal self-assigned this Feb 4, 2022
@krysal krysal added 🟨 priority: medium Not blocking but should be addressed soon and removed 🟧 priority: high Stalls work on the project or its dependents labels Feb 4, 2022
@krysal krysal mentioned this issue Feb 4, 2022
5 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
💻 aspect: code Concerns the software code in the repository 🛠 goal: fix Bug fix 🟨 priority: medium Not blocking but should be addressed soon
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant