-
-
Notifications
You must be signed in to change notification settings - Fork 670
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
[POC] feat: Hono Action #3973
base: main
Are you sure you want to change the base?
[POC] feat: Hono Action #3973
Conversation
Co-authored-by: Yusuke Wada <yusuke@kamawada.com>
This comment has been minimized.
This comment has been minimized.
Bundle size check
Compiler Diagnostics
Reported by octocov |
Codecov ReportAttention: Patch coverage is
❌ Your patch check has failed because the patch coverage (6.54%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #3973 +/- ##
==========================================
- Coverage 91.29% 89.66% -1.63%
==========================================
Files 168 170 +2
Lines 10772 10895 +123
Branches 3057 3029 -28
==========================================
- Hits 9834 9769 -65
- Misses 937 1125 +188
Partials 1 1 ☔ View full report in Codecov by Sentry. |
Hey @usualoma ! Shall we work on Hono Action as just POC? I think Hono Action will be super helpful with using HonoX. In this PR, it throws the following error. Can you take a look at it? App: import { Hono } from '../../src'
import { createAction } from '../../src/jsx/action'
import { jsxRenderer } from '../../src/middleware/jsx-renderer'
const renderer = jsxRenderer(
({ children }) => {
return (
<html>
<body>
<main>{children}</main>
</body>
</html>
)
},
{
stream: true,
}
)
const app = new Hono()
app.use(renderer)
const [action, Component] = createAction(app, async (data, c) => {
return <></>
})
app.get('/', (c) => {
return c.render(
<section>
<form>
<Component />
</form>
</section>
)
})
export default app Error: |
I'll check; give me a few days. |
WIP