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

Added OCPP 2.0.1 feature profiles and link to google sheet #370

Merged
merged 5 commits into from
Jan 13, 2024

Conversation

RobertDeLeeuw
Copy link
Contributor

No description provided.

caller
caller previously requested changes Dec 21, 2023
README.md Outdated

| Feature Profile | Supported |
| -------------------------- | ------------------------- |
| Core | WIP |
Copy link
Contributor

Choose a reason for hiding this comment

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

isnt WIP to weak? since its already in certification? in a way SW is ALWAYS WIP, so I think we reached that point here as well? so maybe say yes? at least thats what we communicating on slides.
smart charging would be WIP if I get JOET correctly..

Copy link
Contributor

Choose a reason for hiding this comment

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

Have just changed both.

I suggest to publish that already as this documentation will all in all be a dynamically changing one in the next weeks / months.

Copy link
Contributor

Choose a reason for hiding this comment

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

@caller Seems like you will have to approve this change (or the whole PR) before we can merge it.

@krealyt krealyt requested a review from caller December 27, 2023 09:41
README.md Show resolved Hide resolved
@krealyt krealyt requested a review from Pietfried January 3, 2024 14:59
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
Co-authored-by: Piet Gömpel <[email protected]>
Signed-off-by: Manuel Ziegler <[email protected]>
| -------------------------- | ------------------------- |
| Core | :heavy_check_mark: yes |
| Advanced Security | WIP |
| Local Auth List Management | |
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
| Local Auth List Management | |
| Local Auth List Management | :heavy_check_mark: yes |

Local Auth List is also already fully implemented

@Pietfried Pietfried dismissed caller’s stale review January 13, 2024 23:13

changes are adressed and approved by others

@Pietfried Pietfried merged commit 5ee114c into main Jan 13, 2024
3 checks passed
@Pietfried Pietfried deleted the rdl-201-readme-update branch January 13, 2024 23:15
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.

5 participants