10796692337

Committed 10 Sep 2024 04:27PM UTC coverage: 67.509% (+0.2%) from 67.325%

Build # 10796692337

Build Type

push

github

Committed by

web-flow

Commit Message

Feat edit account (#1152)

* feat: dynamic account colors

Added options to add a priamary and secondary color when creating an account

Also fixed typo comment

A line in the css wasnt commented properly

* chore: removed experimental code

* refactor: edgecase handling and more

- Handled edge case for fetching color setting when accoutn has no consultant
- wrote test for color computation utils
- type hints additons
- ignored some docker volume folders

* chore: clearing unwanted DB changes

* refactor: code improvements

- type hinting
- segregattion of fucntionalities

The color hex validation is now in a seperate fuciton in the coding_utils file

* chore: removed Alembic comments

* chore: linting code

* fix: removed unwanted classes

* fix: removed unwanted css clases

* chore: initialize repo

* chore: added '#' to default secondary color

* feat: account logo

This feature gives the option to add a logo url to an account

* fix: revert utils.py

accidentally replaced its content with that of another file

* chore: remove unnecessary validation

* chore: typo fix and migration file cleanup

* feat: added extra validation to check logo url length

* feat: logourl fallback

this feature checks te consultant account if the main account desint have a logo set

If both arent set it fallback to the flexmeasures account logo

* fix: url regex validation error

additional pattern was needed to compliment the existing pattern

* refactor: regex patterns for path and port

* refactor: regex patterns for query params

* docs: updated change log with new features changes (white labeling - logo url)

* docs: fixed typo with wrong url linking

* feat: api to edit account details

* refactor: fallback for empty account

- handle edgecase for rendering brand logo incase user is not logged it

* feat: edit accotun popup

- also modified preremdered variables to also send over authto... (continued)

Run Details

6172 of 9926 branches covered (62.18%)

Branch coverage included in aggregate %.

28 of 84 new or added lines in 6 files covered. (33.33%)

1 existing line in 1 file now uncovered.

7832 of 10818 relevant lines covered (72.4%)

5.73 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

8.11

/flexmeasures/utils/validation_utils.py

1	import re	8✔
2
3
4	def validate_color_hex(value):	8✔
5	"""
6	Validates that a given value is a valid hex color code.
7
8	Parameters:
9	:value: The color code to validate.
10	"""
NEW 11	if value is None:	×
NEW 12	return value	×
13
NEW 14	if value and not value.startswith("#"):	×
NEW 15	value = f"#{value}"	×
16
NEW 17	hex_pattern = re.compile(r"^#?([A-Fa-f0-9]{6}\|[A-Fa-f0-9]{3})$")	×
18
NEW 19	if re.match(hex_pattern, value):	×
NEW 20	return value	×
21	else:
NEW 22	raise ValueError(f"{value} is not a valid hex color code.")	×
23
24
25	def validate_url(value):	8✔
26	"""
27	Validates that a given value is a valid URL format using regex.
28
29	Parameters:
30	:value: The URL to validate.
31	"""
NEW 32	if value is None:	×
NEW 33	return value	×
34
NEW 35	url_regex = re.compile(	×
36	r"^(https?\|ftp)://" # Protocol: http, https, or ftp
37	r"((([A-Za-z0-9-]+\.)+[A-Za-z]{2,6})\|" # Domain name
38	r"(\d{1,3}\.){3}\d{1,3})" # OR IPv4
39	r"(:\d+)?" # Port
40	r"(/([A-Za-z0-9$_.+!\'(),;?&=-]\|%[0-9A-Fa-f]{2}))*" # Path
41	r"(\?([A-Za-z0-9$_.+!\'(),;?&=-]\|%[0-9A-Fa-f]{2}))?" # Query string
42	)
43
NEW 44	if not url_regex.match(value):	×
NEW 45	raise ValueError(f"'{value}' is not a valid URL.")	×
46
47	# check if more than 255 characters
NEW 48	if len(value) > 255:	×
NEW 49	raise ValueError(	×
50	"provided logo-url is too long. Maximum length is 255 characters."
51	)
52
NEW 53	return value	×

FlexMeasures / flexmeasures / 10796692337

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous