-
Notifications
You must be signed in to change notification settings - Fork 86
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
Remove Thoth from minimal template #424
Comments
I was wrong. That Fetch method is supplied by the Thoth package. But we don't need to include the Thoth-specific members in the code. In other words, I think that the sample above should still be used - it only uses the raw |
Awesome. Could / should we use text rather than json (given that the message is actually just plain text)? |
I think we should remove Thoth from the minimal template - or at least from the code sample (maybe just keep as a package reference?) - it's simply not required. We also no longer need the use the
json
combinator on the back end:On the server
On the client
The text was updated successfully, but these errors were encountered: