gristlabs / grist-zapier

Code behind the initial Grist/Zapier integration, exported as a reference.
https://support.getgrist.com/zapier/
1 stars 1 forks source link

Zapier can not list available documents #1

Open smalot opened 2 months ago

smalot commented 2 months ago

Hi,

We are facing an issue while listing document on Zapier/Grist connector setup. Both APIKey and endpoint are taken in consideration because projects are correctly listed. May be it's a backword compatibility break on some Grist API endpoints.

Does this widget still maintained ?

BR

dsagal commented 2 months ago

I just tried to reproduce. Zapier the latest version as "Grist (1.2.2)", and seems to work fine at the connection setup stage:

Screenshot 2024-09-02 at 10 06 16 AM

Are you seeing something different? Could you share more information about your setup, i.e. version of Zapier, whether a custom hostname is used, which steps work and which don't, and what errors your are seeing, if any?

paulfitz commented 2 months ago

This repo is an old export from our code on the Zapier platform. It was used as a reference for other integrators at the time it was exported. We don't maintain this export.

I'm leaving a reminder to myself to remove this repo after a grace period to complete talking through this issue.

dsagal commented 2 months ago

Confirming that the current version on Zapier only works for SaaS (i.e. *.getgrist.com). There was an attempt to extend it to self-managed installations, which is why there is a field for the "Hostname" now, but it wasn't finished: only the endpoint to list teams got updated at that time, and everything else still assumes SaaS.

smalot commented 2 months ago

We deployed the release 1.1.17

smalot commented 2 months ago

@paulfitz should I create or update an existing issue on another repository ? possibly we could contribute to fix this issue

dsagal commented 2 months ago

@smalot , I just gave you access (via email) to a private revision of the Zapier integration where I attempted to fix the issue. Would you be able to test it out and let me know if it solves the problem?

smalot commented 2 months ago

I've accepted the invitation, but I'm not sure to know where to find the private revision of the Zapier integration Currently I still see my issue even if I select another team image Where can I find the release of the integration I'm using or select the private revision you provide me ?

dsagal commented 2 months ago

When you go to setup, do you see a version of Grist number 1.2.3?

smalot commented 2 months ago

Indeed, I can use the version 1.2.3 and now I can select the document and table. 👍

image

smalot commented 2 months ago

@dsagal When do you think you can deploy this version in production?

dsagal commented 1 month ago

Hmm, what I am not sure is if we have a way to test that normal Zapier behavior still works correctly in this version. @paulfitz , do you know how to test? The 1.2.3 version does away with the UI-based approach of configuring Zapier, and runs code that's available in this branch https://github.com/gristlabs/grist-zapier/tree/zapier-cli-1.2.3.