Skip to content

feat: regex validation for settings, normailse and lower ThemeColour.#183

Draft
TropicalShadow wants to merge 3 commits into
PteroCA-Org:mainfrom
TropicalShadow:main
Draft

feat: regex validation for settings, normailse and lower ThemeColour.#183
TropicalShadow wants to merge 3 commits into
PteroCA-Org:mainfrom
TropicalShadow:main

Conversation

@TropicalShadow
Copy link
Copy Markdown

fix: #182

Description

implements Regex validation & normalize for setting inputs
Closes: #182

required to update (unsure if this is exactly correct but worked :)

php bin/console doctrine:migrations:migrate --no-interaction --env=prod
php bin/console cache:clear --env=prod

Type of change

  • Bug fix
  • New feature
  • Refactoring / code improvement
  • Documentation update
  • Translation update

Checklist

  • CI pipeline passes (PHPStan, migrations, translations check)
  • Branch targets main
  • Code follows PSR-12 (PHP) / project conventions
  • If adding/editing translations: all language files in src/Core/Resources/translations/ were updated
  • If introducing behavior changes: CHANGELOG entry added

Testing

manual testing to ensure it displays and works
image

- Introduced a new CSS file for dark mode styling to be applied when `forceDarkMode` is enabled.
- Updated `TemplateOptionsDTO` to include `forceDarkMode` option.
- Enhanced logging entity relationships for better data integrity.
- Adjusted user and product price entities to store balance and price as strings for precision.
- Updated Twig templates to conditionally apply dark mode styles and attributes.
- Updated the calculation for memory and disk space in the product template to remove unnecessary division and ensure accurate rounding.
@TropicalShadow
Copy link
Copy Markdown
Author

the last two commits are for a separate thing specific for my own deployment, will probs need to cherry pick out the first commit

@TropicalShadow TropicalShadow marked this pull request as draft March 14, 2026 20:17
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.

Theme colour scheme is not sanity checked

1 participant