Skip to content

Revert to fetch.tools domain#349

Merged
madebyisaacr merged 1 commit into
mainfrom
fetch-tools
Aug 6, 2025
Merged

Revert to fetch.tools domain#349
madebyisaacr merged 1 commit into
mainfrom
fetch-tools

Conversation

@madebyisaacr
Copy link
Copy Markdown
Collaborator

@madebyisaacr madebyisaacr commented Jul 30, 2025

Description

This pull request reverts the OAuth worker domain from framer.wtf to fetch.tools. A few weeks ago fetch.tools couldn't be used, so the team switched to framer.wtf temporarily but now fetch.tools is back.

Currently fetch.tools and framer.wtf are the same.

Testing

Check that these plugins work the same with the new domain:

  • Airtable
  • Search Console
  • Google Sheets
  • HubSpot

Copy link
Copy Markdown
Contributor

@iamakulov iamakulov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ This will break the sign-in flow unless we add back the old (oauth.fetch.tools) redirect_uri in app settings:

  • Airtable
  • Google Search Console App
  • Google Sheets App
  • Hubspot App

Let me try to do that.

@iamakulov
Copy link
Copy Markdown
Contributor

iamakulov commented Jul 30, 2025

Airtable

Hm, Airtable only offers a single redirect_uri (link):

Screenshot 2025-07-30 at 17 47 50

Which means we’ll need to change the URI the moment we deploy the plugin update (and hope the update propagates to everyone right away – does it? or does it take a while?).

Hubspot

Screenshot 2025-07-30 at 17 50 14

✅ Added oauth.fetch.tools

Google

I don’t think I have the right access; @eelco’s out so I’d need @ankon to help updating these:Done

image

@iamakulov
Copy link
Copy Markdown
Contributor

iamakulov commented Jul 30, 2025

All apps except the Airtable one are done ✅

The Airtable one will need to be updated as soon as we deploy the plugin update. Link, creds are in 1Password.

Copy link
Copy Markdown
Contributor

@ankon ankon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See notes by Ivan re Airtable, but code-wise: LGTM.

@madebyisaacr madebyisaacr added this pull request to the merge queue Aug 6, 2025
Merged via the queue into main with commit a525943 Aug 6, 2025
6 of 9 checks passed
@madebyisaacr madebyisaacr deleted the fetch-tools branch August 6, 2025 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants