search

Segelzwerg / django-dynamic-theme

This enables the administrator of a django website to change the theme on the fly.
MIT License
2 stars 2 forks source link

MINOR: input validation for color converter #59

Closed Segelzwerg closed 1 month ago

Segelzwerg commented 2 months ago

After the merge of #41 the color converter requires checks for valid inputs of HEX color strings.

HealthyDoggo commented 2 months ago

How do you want the error to be thrown if an input passed is not a valid HEX color string?

Segelzwerg commented 2 months ago

How do you want the error to be thrown if an input passed is not a valid HEX color string?

Hi @HealthyDoggo thanks for contributing.

How about we create a custom error that is called something NonHexValueError?

HealthyDoggo commented 1 month ago

Sounds good.