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

Fix FirmwareStatusNotificationRequest(status = Installed) (refs EVEREST-1175) #8

Merged
merged 3 commits into from
Sep 4, 2024

Conversation

FaHaGit
Copy link
Contributor

@FaHaGit FaHaGit commented Sep 4, 2024

The variable to publish FirmwareStatusNotification::Installed message is
now delayed, because the message is dropped in case the OCPP client is
not connected or the OCPP module is not yet subscribed to the variable.
This is only a temporary solution to pass TC_L_01 OCPP 2.0.1 test case
and should be removed as soon as a proper fixes are implemented in
the reported issues (EVerest/libocpp#758,
EVerest/everest-core#841).

Signed-off-by: Fabian Hartung <fabian.hartung@chargebyte.com>
@FaHaGit FaHaGit requested a review from barsnick September 4, 2024 08:53
…ST-1175)

The variable to publish FirmwareStatusNotification::Installed message is
now delayed, because the message is dropped in case the OCPP client is
not connected or the OCPP module is not yet subscribed to the variable.
This is only a temporary solution as long as the
reported issues (EVerest/libocpp#758,
EVerest/everest-core#841) are not fixed.

Signed-off-by: Fabian Hartung <fabian.hartung@chargebyte.com>
@FaHaGit FaHaGit force-pushed the Fix_FirmwareStatusNotificationReq_Installed_v2 branch from 409604f to da46425 Compare September 4, 2024 10:19
Signed-off-by: Fabian Hartung <fabian.hartung@chargebyte.com>
@FaHaGit FaHaGit merged commit 9709c13 into main Sep 4, 2024
@FaHaGit FaHaGit deleted the Fix_FirmwareStatusNotificationReq_Installed_v2 branch September 4, 2024 10:57
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.

2 participants