Conversation
|
Someone is attempting to deploy a commit to the cal Team on Vercel. A member of the Team first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
this is super dope! great setup screen (cc @Jaibles – we should use this for other apps too) and great loom! cant wait to have this go live. do we also plan to have actions? or only triggers for now |
|
also added you to the calcom organisation so you can now create branches from within the repo vs from the outside fork |
zomars
left a comment
There was a problem hiding this comment.
Some comments and updates. Overall this is very well done. Just making some adjustments to match our new way to handle app installations.
| api_key: process.env.GIPHY_API_KEY, | ||
| }); | ||
| } | ||
| await createApp("zapier", "zapier", ["other"], "zapier_other"); |
There was a problem hiding this comment.
This is so the app is actually added to the DB
packages/prisma/schema.prisma
Outdated
| hashedKey String @unique() | ||
| user User? @relation(fields: [userId], references: [id], onDelete: Cascade) | ||
| hashedKey String @unique() | ||
| apiKeyType ApiKeyType @default(OTHER) |
There was a problem hiding this comment.
BTW we now have an App model. It theory we could just add a relationship like: appId: "zapier" that way we could have a way to fetch only zapier related api keys without having add a new type.
| "generate_api_key": "Generate Api Key", | ||
| "your_unique_api_key": "Your unique API key", | ||
| "copy_safe_api_key": "Copy this API key and save it somewhere safe. If you lose this key you have to generate a new one.", | ||
| "zapier_setup_instructions": "<0>Log into your Zapier account and create a new Zap.</0><1>Select Cal.com as your Trigger app. Also choose a Trigger event.</1><2>Choose your account and then enter your Unique API Key.</2><3>Test your Trigger.</3><4>You're set!</4>", |
There was a problem hiding this comment.
Consolidated setup instructions to a single string.
| "example_name": "Juan Pérez", | ||
| "you_are_being_redirected": "Serás redirigido a {{ url }} en $t(second, {\"count\": {{seconds}} })." | ||
| "you_are_being_redirected": "Serás redirigido a {{ url }} en $t(second, {\"count\": {{seconds}} }).", | ||
| "zapier_setup_instructions": "<0>Inicia sesión en tu cuenta de Zapier y crea un nuevo Zap.</0><1>Selecciona Cal.com cómo tu aplicación disparadora. Tambien elige tu evento disparador.</1><2>Elige tu cuenta e ingresa tu Clave API única.</2><3>Prueba tu disparador.</3><4>¡Listo!</4>" |
There was a problem hiding this comment.
Also added Spanish translation to make sure it works
What does this PR do?
Adds Zapier App to App Store
Demo
https://www.loom.com/share/1f7bc75564014e0fa3e4415b2d416d40
fixes #1543
Type of change
How should this be tested?
Checklist