content: say that OCPP chargers need no driver, and link the guide - #8
Open
HuggeK wants to merge 1 commit into
Open
content: say that OCPP chargers need no driver, and link the guide#8HuggeK wants to merge 1 commit into
HuggeK wants to merge 1 commit into
Conversation
FTW ships an OCPP 1.6J + 2.0.1 central system, so a charger that speaks either connects with no Lua driver. The site never said so — a visitor asking "do I need a driver for my charger?" had no way to find out. Folded into the existing integrations card rather than a seventh one: the capability grid is repeat(3, 1fr), so a seventh card would leave a single card stranded on its own row. Co-authored-by: HuggeK <48095810+HuggeK@users.noreply.github.com>
miravoss26
reviewed
Aug 5, 2026
miravoss26
left a comment
There was a problem hiding this comment.
Marketing copy only: adds OCPP language to the meta description/keywords, the EV-charging map node label, the "Open integrations" capability copy, and a new "skip the driver" bullet + link to srcfl/ftw/docs/ocpp.md. No code, no config, no build files touched.
- Content checks out against what already shipped (OCPP support in ftw core) — not overselling something unbuilt.
- The new docs link (
github.com/srcfl/ftw/blob/master/docs/ocpp.md) — worth a human eyeballing that it resolves; I can't fetch it from here. - No security surface: static HTML copy edit, no scripts/attributes changed.
Safe to merge from my read, modulo confirming that one doc link resolves.
This file contains hidden or 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
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.
FTW ships a built-in OCPP Central System (1.6J and 2.0.1), so an EV charger that speaks either connects directly and needs no Lua driver. The site never said so anywhere — a visitor asking "do I need a driver for my charger?" had no way to find the answer, and "OCPP" did not appear on the page at all.
This adds the claim where it is actually load-bearing, and links
docs/ocpp.md.What changed
HTTP · MQTT · OCPP · Lua.OCPP / V2Xinstead ofload / V2X.<meta>description and keywords — OCPP, OCPP 1.6J, OCPP 2.0.1 added, so the page is findable by people searching for exactly this.Why this is not a seventh capability card
.capability-gridisgrid-template-columns: repeat(3, 1fr)with border-drawn cells, and there are currently six cards — two clean rows of three. A seventh would strand one card alone on row three (and alone again in the two-column breakpoint atmax-widthwhere it becomes1fr 1fr). Going to nine would mean inventing two capabilities that do not exist.Folding the fact into the card that already covers integrations keeps the grid rhythm intact and puts OCPP next to HTTP and MQTT, where it belongs.
Note on card height
Card 06's paragraph grows by roughly one line.
.capability-grid articleismin-height: 300pxwithcode { margin-top: auto }, so the card absorbs it and the row sizes to its tallest member; rows are independent, so row 1 is unaffected. The sentence was deliberately kept short for this reason.Review note
This is a visual change and I have no browser in this environment, so the layout reasoning above is from the CSS rather than from a render. Please eyeball the capabilities grid and the Build section before merging — particularly that the three
.inline-actionslinks still sit on one line at desktop width.Upstream context: srcfl/ftw#732 adds the OCPP server this describes.
🤖 Generated with Claude Code