fix: use callback url in signup (#6010)
* fix: use callback url in signup Signed-off-by: Udit Takkar <udit.07814802719@cse.mait.ac.in> * fix: do login after sign up and redirect to onboarding Signed-off-by: Udit Takkar <udit.07814802719@cse.mait.ac.in> * chore: add telemetry Signed-off-by: Udit Takkar <udit.07814802719@cse.mait.ac.in> * fix: error handling Signed-off-by: Udit Takkar <udit.07814802719@cse.mait.ac.in> Signed-off-by: Udit Takkar <udit.07814802719@cse.mait.ac.in> Co-authored-by: Peer Richelsen <peeroke@gmail.com> Co-authored-by: Omar López <zomars@me.com>pull/6069/head^2
parent
916468b8cc
commit
df0a522656
|
@ -6,6 +6,7 @@ import { FormProvider, SubmitHandler, useForm } from "react-hook-form";
|
||||||
import LicenseRequired from "@calcom/features/ee/common/components/v2/LicenseRequired";
|
import LicenseRequired from "@calcom/features/ee/common/components/v2/LicenseRequired";
|
||||||
import { isSAMLLoginEnabled } from "@calcom/features/ee/sso/lib/saml";
|
import { isSAMLLoginEnabled } from "@calcom/features/ee/sso/lib/saml";
|
||||||
import { useLocale } from "@calcom/lib/hooks/useLocale";
|
import { useLocale } from "@calcom/lib/hooks/useLocale";
|
||||||
|
import { collectPageParameters, telemetryEventTypes, useTelemetry } from "@calcom/lib/telemetry";
|
||||||
import { inferSSRProps } from "@calcom/types/inferSSRProps";
|
import { inferSSRProps } from "@calcom/types/inferSSRProps";
|
||||||
import { Alert, Button, EmailField, PasswordField, TextField } from "@calcom/ui";
|
import { Alert, Button, EmailField, PasswordField, TextField } from "@calcom/ui";
|
||||||
import { HeadSeo } from "@calcom/web/components/seo/head-seo";
|
import { HeadSeo } from "@calcom/web/components/seo/head-seo";
|
||||||
|
@ -26,6 +27,8 @@ type FormValues = {
|
||||||
export default function Signup({ prepopulateFormValues }: inferSSRProps<typeof getServerSideProps>) {
|
export default function Signup({ prepopulateFormValues }: inferSSRProps<typeof getServerSideProps>) {
|
||||||
const { t } = useLocale();
|
const { t } = useLocale();
|
||||||
const router = useRouter();
|
const router = useRouter();
|
||||||
|
const telemetry = useTelemetry();
|
||||||
|
|
||||||
const methods = useForm<FormValues>({
|
const methods = useForm<FormValues>({
|
||||||
defaultValues: prepopulateFormValues,
|
defaultValues: prepopulateFormValues,
|
||||||
});
|
});
|
||||||
|
@ -53,8 +56,12 @@ export default function Signup({ prepopulateFormValues }: inferSSRProps<typeof g
|
||||||
})
|
})
|
||||||
.then(handleErrors)
|
.then(handleErrors)
|
||||||
.then(async () => {
|
.then(async () => {
|
||||||
await signIn("Cal.com", {
|
telemetry.event(telemetryEventTypes.login, collectPageParameters());
|
||||||
callbackUrl: router.query.callbackUrl ? `${WEBAPP_URL}/${router.query.callbackUrl}` : WEBAPP_URL,
|
await signIn<"credentials">("credentials", {
|
||||||
|
...data,
|
||||||
|
callbackUrl: router.query.callbackUrl
|
||||||
|
? `${WEBAPP_URL}/${router.query.callbackUrl}`
|
||||||
|
: `${WEBAPP_URL}/getting-started`,
|
||||||
});
|
});
|
||||||
})
|
})
|
||||||
.catch((err) => {
|
.catch((err) => {
|
||||||
|
@ -111,7 +118,9 @@ export default function Signup({ prepopulateFormValues }: inferSSRProps<typeof g
|
||||||
className="w-5/12 justify-center"
|
className="w-5/12 justify-center"
|
||||||
onClick={() =>
|
onClick={() =>
|
||||||
signIn("Cal.com", {
|
signIn("Cal.com", {
|
||||||
callbackUrl: (`${WEBAPP_URL}/${router.query.callbackUrl}` || "") as string,
|
callbackUrl: router.query.callbackUrl
|
||||||
|
? `${WEBAPP_URL}/${router.query.callbackUrl}`
|
||||||
|
: `${WEBAPP_URL}/getting-started`,
|
||||||
})
|
})
|
||||||
}>
|
}>
|
||||||
{t("login_instead")}
|
{t("login_instead")}
|
||||||
|
|
Loading…
Reference in New Issue