-
Notifications
You must be signed in to change notification settings - Fork 173
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
Ability to delete a prompt #2
Comments
Can I work on this @arielweinberger ? |
@alceil Of course! Go for it. |
Since there's no progress with this one, I'll leave it to somebody else from the community for now. |
@AyanDas348 would you like to take this? It's quite simply - add a service method to Then add a The Delete button should be displayed as follows: There's already a prompt deletion confirmation modal implemened (check out p.s. you'll need to adjust the Prisma Schema to cascade delete all dependants of a Prompt when you delete it. You can see how to do that here. |
@arielweinberger Sure I'll give it a try. Thanks! |
Fixed in #84, to be released very soon in |
We need to have the ability to delete a prompt.
add a service method to
prompts.service.ts
to delete the prompt. You'l use Prisma to delete it (look at the other methods in the service).Then add a
deletePrompt
mutation to theprompts.resolver.ts
.The Delete button should be displayed as follows:
There's already a prompt deletion confirmation modal implemened (check out
apps/console/src/app/pages/prompts/[promptId]/index.tsx
line 69). So all in all it's a fun task.p.s. you'll need to adjust the Prisma Schema to cascade delete all dependants of a Prompt when you delete it. You can see how to do that here.
The text was updated successfully, but these errors were encountered: