Feature: DeleteCertificates message and Installed Certificates CRUD #21
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 3 commits
October 18, 2024 23:35
…rtificate hashes for installed certificates that are on the chargers feat: because there are some issues with creation, for now enabled the primary field when creating a new record, because otherwise there was no way to create a charging station, also enabled edit in /evses/id/edit for now to try and step around the bugs a bit to make more functional progress, most bugs related to associations or editing not quite populating correctly and persisting in some cases. feat: created initial DeleteCertificate message component and added it to charging station ui, and then also created a custom action to delete certificate that is also in the InstallCertificate table UI
…eature/delete-certificate * feature/making-message-components-more-reusable: refactor: update reset to use generic form and correct EVSE selection. bugfix: fixing linting in trigger-message bugfix: removing unused imports bugfix: only including evse in trigger message if available as an argument + fixing response handling when sending trigger message request bugfix: fixing linting in trigger-message bugfix: removing unused imports bugfix: only including evse in trigger message if available as an argument + fixing response handling when sending trigger message request
…eature/delete-certificate * feature/making-message-components-more-reusable: chore: resolving build and lint issues
thanaParis
changed the base branch from
feature/making-message-components-more-reusable
to
rc-0.2.0
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Please Note: Related CORE PR citrineos/citrineos-core#284
Please Note: Because of issue with association selection, in some of the screenshots, you will see the selection doesnt show the value properly, but the component does have it in memory and the request does complete successfully!