-
Notifications
You must be signed in to change notification settings - Fork 281
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Resolves the issue regarding copy link in form #53
base: main
Are you sure you want to change the base?
Conversation
Thank you for following the naming conventions! 🙏 |
components/Dashboard/FormPane.tsx
Outdated
@@ -3,7 +3,7 @@ import Image from 'next/image' | |||
import Link from 'next/link' | |||
import { useRouter } from 'next/router' | |||
import { useState } from 'react' | |||
|
|||
import toast from 'react-hot-toast' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Move the import statement above as we're separating our imports from built-in libs to custom libs
Sure, @hemantwasthere. I am working on it |
I have implemented the suggestion, please review |
components/Dashboard/FormPane.tsx
Outdated
import Link from 'next/link' | ||
import { useRouter } from 'next/router' | ||
import { useState } from 'react' | ||
import toast from 'react-hot-toast' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
put a space of a line between the imports
like this:
import Link from 'next/link'
import { useRouter } from 'next/router'
import { useState } from 'react'
// space here
import { Checkbox } from '@/components/ui/checkbox'
import {
Tooltip,
TooltipContent,
TooltipProvider,
TooltipTrigger,
} from '@/components/ui/tooltip'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done sir
gql/graphql.ts
Outdated
@@ -73,6 +73,28 @@ export type Form = { | |||
updatedAt?: Maybe<Scalars['Date']['output']>; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Discard all the other file changes except FormPane.tsx
Hey there, there is a merge conflict, can you take a look? |
What does this PR do?
This PR fix the copy button to copy the link of the forms.
Fixes #52
Type of change
Checklist