Skip to content

Conversation

@RobertDeLeeuw
Copy link
Contributor

No description provided.

Signed-off-by: Robert de Leeuw, PIONIX <robert@pionix.de>
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.

Signed-off-by: Krealyt <manuel.ziegler@pionix.de>
Signed-off-by: Krealyt <manuel.ziegler@pionix.de>
@krealyt krealyt requested a review from caller December 27, 2023 09:41
Signed-off-by: Krealyt <manuel.ziegler@pionix.de>
@krealyt krealyt requested a review from Pietfried January 3, 2024 14:59
Co-authored-by: Piet Gömpel <37657534+Pietfried@users.noreply.github.com>
Signed-off-by: Manuel Ziegler <113091917+krealyt@users.noreply.github.com>
| -------------------------- | ------------------------- |
| 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
@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.

6 participants