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.

Sign up for GitHub

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

Jump to bottom

Add method to send cleared charging limit request #62

Merged
merged 10 commits into from
Oct 8, 2024
Merged

Add method to send cleared charging limit request #62

merged 10 commits into from
Oct 8, 2024

Conversation

couryrr-afs
Copy link

Describe your changes

  • added smart_charging_handler method to evaluate request and set up appropriate request for CSMS
  • added public charge_point method on_external_limit_cleared for use when an external system clears a constraint.

Issue ticket number and link

Checklist before requesting a review

@couryrr-afs couryrr-afs force-pushed the couryrr/send-cleared-charging-limit-request branch from 789d6fe to 769cc0a Compare October 8, 2024 15:55
@couryrr-afs
smartcharging: added handler for clearing external constraint
34ef29e 
Signed-off-by: Coury Richards <146002925+couryrr-afs@users.noreply.github.com>
@couryrr-afs
smartcharging: updated return to handle both request types
bf34867 
Signed-off-by: Coury Richards <146002925+couryrr-afs@users.noreply.github.com>
@couryrr-afs
smartcharging: added logic to check limit change significance and sen…
a1d344b 
…d TransactionEventRequest

Signed-off-by: Coury Richards <146002925+couryrr-afs@users.noreply.github.com>
@couryrr-afs
charge point: added public method to notify csms when external limit …
cec055c 
…is cleared

Signed-off-by: Coury Richards <146002925+couryrr-afs@users.noreply.github.com>
@couryrr-afs
smart charging: updated transaction even with additional attributes b…
9481cf2 
…ut some details are still needed.

Signed-off-by: Coury Richards <146002925+couryrr-afs@users.noreply.github.com>
@couryrr-afs
updated: using vector to pass back multiple TransactionEventRequest.
65b4044 
Signed-off-by: Coury Richards <146002925+couryrr-afs@users.noreply.github.com>
@couryrr-afs
updated: added optional evse_id to on_handle method
8a2e366 
Signed-off-by: Coury Richards <146002925+couryrr-afs@users.noreply.github.com>
@couryrr-afs
updated: added optional evse_id to handler and passed
d66c9ac 
Signed-off-by: Coury Richards <146002925+couryrr-afs@users.noreply.github.com>
@couryrr-afs
updated: check for open transaction on an evse
ec25f77 
Signed-off-by: Coury Richards <146002925+couryrr-afs@users.noreply.github.com>
@couryrr-afs
updated: moved duplicated code to method
084a62c 
Signed-off-by: Coury Richards <146002925+couryrr-afs@users.noreply.github.com>
@couryrr-afs couryrr-afs force-pushed the couryrr/send-cleared-charging-limit-request branch from 69e604d to 084a62c Compare October 8, 2024 19:30
@couryrr-afs couryrr-afs merged commit 60226e9 into feature/k12-set-update-external-limits Oct 8, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gberardi-pillar gberardi-pillar gberardi-pillar approved these changes

@christopher-davis-afs christopher-davis-afs christopher-davis-afs approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@couryrr-afs @gberardi-pillar @christopher-davis-afs