-
-
Notifications
You must be signed in to change notification settings - Fork 360
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
Issue: URL for published form throws an error #845
Comments
Thanks for reporting @nsmathew. Could you share the link that was copied to your clipboard, excluding/censoring the
|
Sure, it is |
Thanks. Shape of the URL looks right to me. Any chance you see an error trace in the server logs? |
Thanks for the screenshot. A few lines just before the first line in your screenshot, there should be a line starting with |
Had a look but I don't see any such lines in the docker logs. |
We have the same issue. Form Preview works and populates the table but the published link (https://mydomain.com/forms//1) returns a 400 error. It's self-hosted, behind an apache reverse proxy, we encounter no other issues. I'll post further debugging logs if it helps anyone. A minor difference: the error message is "Fetch failed". |
@gdedrouas I also have encountered this issue in a local environment (in our production though we have a different issue). That's something I meant to fix through this issue at first: https://github.com/gristlabs/grist-core/pull/864/files It would mean that you would have to set a new env variable (at least in my case, it would solve my issue): But I was told that the guilty piece of code requiring that would probably be changed soon so this new env variable would not be necessary. So probably the PR I can close this PR @paulfitz? |
I can confirm that the request to "/api/s..." is never made. getHomeUrl fails in our environment. |
Getting same issue here in local environment, however, unlike @gdedrouas, I can view the preview form and submit, but it does not populate the table. |
The next sync to |
The sync of grist-saas code @georgegevoian mentioned happened this morning. The relevant change I think is this one: Maybe @georgegevoian can give more context. Sorry for how opaque this change was, we are slowly moving more development over to grist-core but it is still a work in progress. |
The It's part of a broader refactor of forms, to more closely align with the other client-side Grist code. All forms API requests are now routed through DocApiForwarder, which should do a better job at building URLs that the home server can actually reach. I'm not sure if it'll address all of the issues (e.g. the submission issue @StreamlinedStartup reported, which may be unrelated), but please give it a try and let us know how it fares. |
It fixes the Though the problem is different from the |
It also fixes the Fetch failed error we got. Much appreciated. |
Function/Service
Forms
Issue
Updated to the latest version of grist today. It is running as a container in my self-hosted server. I tried the new forms functionality and created a form and published it. The link generated as part of the preview works fine, the data I key in gets saved in the appropriate table. However, when I try the link from the 'Copy Link' option in the form I get a page as shown in the screengrab.
Error message
There was an error: Unexpected token < in JSON at position 0.
Screenshot
The text was updated successfully, but these errors were encountered: