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

Add user configuration management and localization features #220

Draft
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

oriolreverte36
Copy link
Contributor

Introduce an AccountController for user data management, enhance the AccountConfig view with personal information and Spanish localization, and implement functionality for user updates and password management. Clean up the code and improve UI consistency with disabled states for inputs and buttons.

@github-actions github-actions bot added kind/feature Categorizes issue or PR as related to a new feature. kind/design Categorizes issue or PR as related to design. area/app/views Categorizes issue or PR as related to app/views. area/app/controllers Categorizes issue or PR as related to app/controllers. area/routes Categorizes issue or PR as related to routes or router. area/javascript Categorizes issue or PR as related to JavaScript. labels Dec 17, 2024
Copy link

codecov bot commented Dec 17, 2024

Codecov Report

Attention: Patch coverage is 0% with 45 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...pp/src/app/Controllers/Admin/AccountController.php 0.00% 30 Missing ⚠️
...rc/app/Views/Admin/AccountConfig/AccountConfig.php 0.00% 14 Missing ⚠️
app/src/app/Layouts/Admin/AdminLayout.php 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              main    #220      +/-   ##
==========================================
- Coverage     4.66%   4.59%   -0.07%     
- Complexity     306     310       +4     
==========================================
  Files           93      95       +2     
  Lines         2892    2936      +44     
==========================================
  Hits           135     135              
- Misses        2757    2801      +44     
Files with missing lines Coverage Δ Complexity Δ
app/src/app/Controllers/Auth/AuthController.php 0.00% <ø> (ø) 10.00 <0.00> (ø)
app/src/app/Layouts/Admin/AdminLayout.php 0.00% <0.00%> (ø) 0.00 <0.00> (ø)
...rc/app/Views/Admin/AccountConfig/AccountConfig.php 0.00% <0.00%> (ø) 0.00 <0.00> (?)
...pp/src/app/Controllers/Admin/AccountController.php 0.00% <0.00%> (ø) 4.00 <4.00> (?)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/app/controllers Categorizes issue or PR as related to app/controllers. area/app/views Categorizes issue or PR as related to app/views. area/javascript Categorizes issue or PR as related to JavaScript. area/routes Categorizes issue or PR as related to routes or router. kind/design Categorizes issue or PR as related to design. kind/feature Categorizes issue or PR as related to a new feature.
Projects
Status: Pendent
Development

Successfully merging this pull request may close these issues.

2 participants