Skip to content

Feature: Routing Forms Typeform App and other improvements#3625

Merged
kodiakhq[bot] merged 68 commits into
mainfrom
improvements/routing-forms
Aug 13, 2022
Merged

Feature: Routing Forms Typeform App and other improvements#3625
kodiakhq[bot] merged 68 commits into
mainfrom
improvements/routing-forms

Conversation

@hariombalhara
Copy link
Copy Markdown
Member

@hariombalhara hariombalhara commented Aug 1, 2022

What does this PR do?

Demo Loom

Fixes #3619 #3559

Environment: Production

Type of change

  • Improvements

How should this be tested?

  • Go and create a routing form you would be able to see the new creation flow with dialog
  • You would be able to duplicate a form with routes and fields in it.
  • You would be able to get the code to Embed a form
  • You won't be able to add rules to Fallback Route. But if there are any rules already in it, you would still see them. Once you delete them, you won't see add rule option in Fallback Route again.
  • Form deletion would now ask for confirmation

@vercel
Copy link
Copy Markdown

vercel Bot commented Aug 1, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
cal 🔄 Building (Inspect) Aug 13, 2022 at 10:56AM (UTC)
cal-com ❌ Failed (Inspect) Aug 13, 2022 at 10:56AM (UTC)
nightly-cal ❌ Failed (Inspect) Aug 13, 2022 at 10:56AM (UTC)
1 Ignored Deployment
Name Status Preview Updated
swagger ⬜️ Ignored (Inspect) Aug 13, 2022 at 10:56AM (UTC)

@hariombalhara hariombalhara added the ♻️ autoupdate tells kodiak to keep this branch up-to-date label Aug 1, 2022
@hariombalhara hariombalhara marked this pull request as ready for review August 1, 2022 13:16
@hariombalhara hariombalhara force-pushed the improvements/routing-forms branch from b4024ae to ce0e21b Compare August 1, 2022 13:32
Comment thread tests/config/playwright.config.ts
Comment thread packages/app-store/_baseApp/api/add.ts
Comment on lines 58 to 62
<div className="mr-5">
<img className="h-11" src="/api/app-store/zapier/icon.svg" alt="Zapier Logo" />
<img className="h-11" src="/api/app-store/typeform/icon.svg" alt="Zapier Logo" />
</div>
<div className="ml-5">
<div className="text-gray-600">{t("setting_up_zapier")}</div>
<div className="text-gray-600">How to integrate a Typeform with Routing Form</div>
{!newApiKey ? (
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is still on zapier/pages is this intended?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You didn't answer my question before resolving. Is putting the typeform text and icon in the Zapier page and intended change? @hariombalhara

Comment thread yarn.lock Outdated
Comment on lines +19653 to +19657
react-hook-form@^7.16.2:
version "7.34.0"
resolved "https://registry.yarnpkg.com/react-hook-form/-/react-hook-form-7.34.0.tgz#22883b5e014e5c5e35f3061d0e3862153b0df2ec"
integrity sha512-s0/TJ09NVlEk2JPp3yit1WnMuPNBXFmUKEQPulgDi9pYBw/ZmmAFHe6AXWq73Y+kp8ye4OcMf0Jv+i/qLPektg==

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We already have 7.31.1 below, let's try to match that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

app-store area: app store, apps, calendar integrations, google calendar, outlook, lark, apple calendar automerge core area: core, team members only

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

Improvements: Routing Form Routing Forms: Disable rules in Fallback Route

3 participants