Documentation example for `server_component.include` does not type check

[justinlubin]

The example for server_component.include includes the following snippet:

html.button(
  [server_component.include(["target.id"]), event.on("click", handler)],
  [html.text("Click me!")],
)

However, server_component.include takes in two arguments. Therefore, I believe the code snippet should be as follows:

html.button(
  [server_component.include(event.on("click", handler), ["target.id"])],
  [html.text("Click me!")],
)

or something similar but with a pipeline.

I'm happy to make a PR fixing this—just let me know the preferred code style!

[hayleigh-dot-dev]

Ah that's a good spot! We changed this api when we released v5 a while back and it must've slipped through the cracks!

I think the recommended style would be to pipe into include the same way we recommend piping into debounce or throttle, so:

html.button(
  [
    event.on("click", handler)
    |> server_component.include(["target.id"])
  ],
  [html.text("Click me!")
)

If you wanted to make a PR that'd be fab, but don't worry about it if you don't have time!