Skip to content

Feat: Charging Site status override - 3793#4013

Merged
hamed-valiollahi merged 5 commits intodevelopfrom
feat/hamed-charging-site-status-override-3793
Mar 12, 2026
Merged

Feat: Charging Site status override - 3793#4013
hamed-valiollahi merged 5 commits intodevelopfrom
feat/hamed-charging-site-status-override-3793

Conversation

@hamed-valiollahi
Copy link
Collaborator

This PR introduces role-based manual status override for Charging Sites.

Closes #3793

@github-actions
Copy link

github-actions bot commented Mar 10, 2026

Backend Test Results

1 789 tests   1 788 ✅  14m 1s ⏱️
   45 suites      1 💤
    1 files        0 ❌

Results for commit df50478.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Mar 10, 2026

Frontend Test Results

    1 files    315 suites   3m 27s ⏱️
5 178 tests 5 160 ✅ 18 💤 0 ❌
5 188 runs  5 170 ✅ 18 💤 0 ❌

Results for commit df50478.

♻️ This comment has been updated with latest results.

Copy link
Collaborator

@AlexZorkin AlexZorkin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good but one enum fix would be good



class ChargingSiteManualStatusUpdateSchema(BaseSchema):
new_status: str
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should be an enum value

@hamed-valiollahi
Copy link
Collaborator Author

looks good but one enum fix would be good

Thanks for the review. Of course! I’ve applied the fix.

Copy link
Collaborator

@AlexZorkin AlexZorkin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@hamed-valiollahi hamed-valiollahi merged commit 0a39ee4 into develop Mar 12, 2026
1 check passed
AlexZorkin pushed a commit that referenced this pull request Mar 13, 2026
…verride-3793

Feat: Charging Site status override - 3793
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.

LCFS - Charging Site manual status change

2 participants