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

[Bug] - Fix Type error in Colors.vue without default colors #837

Merged
merged 2 commits into from
Feb 12, 2024

Conversation

shahzadbinshahjahan
Copy link
Contributor

Issue

When adding a color field by customizing the form without specifying default values, Color.vue throws a Type error because it tries to find the color from an undefined colors array

Changes Made

Added a condition before the find call to return the value if there are no items in the colors array.
Screenshot from 2024-02-02 18 23 29
Screenshot from 2024-02-02 18 21 44

@Isaac-GC
Copy link
Collaborator

Isaac-GC commented Feb 8, 2024

Hi @shahzadbinshahjahan Can you fix the linting issue?

Should just be able to run yarn prettier --write src/components/Controls/Color.vue and be good

@shahzadbinshahjahan
Copy link
Contributor Author

Hi @Isaac-GC, I have fixed the linting issue. Thank you for your help.

@Isaac-GC Isaac-GC merged commit 71c4e8a into frappe:master Feb 12, 2024
4 checks passed
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.

None yet

2 participants