-
-
Notifications
You must be signed in to change notification settings - Fork 182
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
204 improvements #68
204 improvements #68
Conversation
This looks great @theeyed! Do you mind adding a test for Then I think this one is good to go! |
Hey @theeyed, let me know if you don't have time to wrap this up. I don't want this to get stale. |
@matthewmueller got the original test passing. It required adding |
Excellent, thank you Alex! The Svelte version is needed because it starts off with a fresh node_modules directory every test run and this test depends on Svelte. |
* return 204 on nil error response * allow void view actions * fix controller test for 204 nil error response * Fix `TestEmptyActionWithView`
* return 204 on nil error response * allow void view actions * fix controller test for 204 nil error response * Fix `TestEmptyActionWithView`
Fixes #10
nil
error responsesmap[string]interface{}{}
to view Handler