Broken Telefunc boilerplate?

[brillout]

Describe the bug

The generated boilerplate for Telefunc seems broken.

To Reproduce

pnpm create bati --telefunc --hono

Expected behavior

A boilerplate that kind of make sense.

[brillout]

Ah, I omitted React/Vue/Solid that's maybe why. Let me try with a UI framework.

[brillout]

With React, the generated boilerplate is quite good. Although it'd be nice if pages/todo/TodoList.telefunc.ts > onNewTodo() mutates the database instead of just printing a message.

[magne4000]

Related to #263

With React, the generated boilerplate is quite good. Although it'd be nice if pages/todo/TodoList.telefunc.ts > onNewTodo() mutates the database instead of just printing a message.

It does if you select a DB like sqlite or drizzle

[brillout]

I'd like to use pnpm create bati --{react,vue,solid} --telefunc --hono for creating repos that will be linked next to the Vike Talk. It'b nice to have something minimal that reproduces Next.js.

[magne4000]

I'd like to use pnpm create bati --{react,vue,solid} --telefunc --hono for creating repos that will be linked next to the Vike Talk. It'b nice to have something minimal that reproduces Next.js.

What would you need that is not available?

[brillout]

What would you need that is not available?

I think it's perfect! Just did a bit of polishing, see my PRs.

[brillout]

The only thing is that I'm thinking maybe we really should be removing that plain Vike boilerplate, it's quite a frankenstein :) But I don't know whether it's a lot of effort to remove 👀

[magne4000]

I will remove it, it's half-baked and doesn't serve any real purpose. I just need to find a way to adapt the UI (I'll probably pre-select react)