Skip to content
This repository has been archived by the owner on Nov 21, 2022. It is now read-only.

Please improve the readme on webhooks #9

Open
doc-l opened this issue Nov 12, 2020 · 2 comments
Open

Please improve the readme on webhooks #9

doc-l opened this issue Nov 12, 2020 · 2 comments

Comments

@doc-l
Copy link

doc-l commented Nov 12, 2020

It contains syntax errors:

app.use("/events", EventsHandler(events: []Event) => {

should be

app.use("/events", EventsHandler(events: Event[]) => {

and

app.use("/sign_transaction", SignTransactionHandler(req: SignTransactionRequest, resp: SignTransactionResponse) => {
    // decide whether or not to sign() or reject() the request.
}, secret),

has a missing ( right after SignTransactionHandler

@chaseeb
Copy link

chaseeb commented Jan 19, 2021

was this resolved?

@doc-l
Copy link
Author

doc-l commented Jan 20, 2021

no, the readme still contains those syntax errors

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants