Skip to content
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

Option to delete draft projects #1271

Open
laurenluz opened this issue Aug 2, 2022 · 28 comments
Open

Option to delete draft projects #1271

laurenluz opened this issue Aug 2, 2022 · 28 comments
Assignees

Comments

@laurenluz
Copy link
Member

I have so many sketchy test drafts in "my projects" with no clear way to get rid of them? Can we add a little trashcan w/ an "are you sure?" modal please?

@mosaeedi @MoeNick

image.png

@divine-comedian
Copy link
Contributor

image
Add Delete Project as option for actions menu when project is in draft mode.

We should add a confirmation modal ARE YOU SURE YOU WANT TO DELETE YOUR PROJECT?

@laurenluz
Copy link
Member Author

omg guys this is getting painful... @divine-comedian @mosaeedi

there was an update to "My projects" where now the projects info takes up so much more space on the page...

and I still can't delete all the useless drafts which all appear FIRST. above my very legitimate projects.. Giveth & the matching pool.

so "my projects" is just now a total mess for me, and I'm sure I'm not the only one.. can we please prioritize this issue?

delete.draft.mov

@divine-comedian divine-comedian added P2 A normal to do item or feature. and removed P3 nice to have, but not required labels May 1, 2024
@mosaeedi
Copy link
Collaborator

mosaeedi commented May 1, 2024

Link to Figma

When a project is in draft mode, the Delete Project option should be added to the action dropdown.

And when users click the Delete option, we need to make sure they know what they are doing with a confirmation modal.

Screenshot 2024-05-01 at 2 31 19 PM

@divine-comedian
Copy link
Contributor

This looks great! I made Giveth/impact-graph#1519 to follow up making the back-end endpoint for this before proceeding with FE

@laurenluz
Copy link
Member Author

you guys are amazing <3 thank you

@MohammadPCh
Copy link
Collaborator

@mohammadranjbarz Do we have an API to delete draft projects?

@mohammadranjbarz
Copy link
Contributor

@mohammadranjbarz Do we have an API to delete draft projects?

No we don't have any API for deleting projects (draft or non-draft)
@divine-comedian @laurenluz if we need it, please create an issue for that

@divine-comedian
Copy link
Contributor

divine-comedian commented Jun 23, 2024

This looks great! I made Giveth/impact-graph#1519 to follow up making the back-end endpoint for this before proceeding with FE

@mohammadranjbarz - issue was made here about 2 months ago, but it was never assigned to anyone 😅

@mohammadranjbarz
Copy link
Contributor

This looks great! I made Giveth/impact-graph#1519 to follow up making the back-end endpoint for this before proceeding with FE

@mohammadranjbarz - issue was made here about 2 months ago, but it was never assigned to anyone 😅

unagi

@MohammadPCh
Copy link
Collaborator

@mohammadranjbarz Is there any new update for it?

@mohammadranjbarz
Copy link
Contributor

@mohammadranjbarz Is there any new update for it?

I just assigned it to @Reshzera , he will let you know here when implemented and merged

@divine-comedian
Copy link
Contributor

@MohammadPCh - @Reshzera has implemented the BE endpoint, Is there another FE dev we can pass this to?

@Reshzera please share any details on accessing the endpoint

@divine-comedian
Copy link
Contributor

the backend has been merged in https://github.com/orgs/Giveth/projects/9/views/1?filterQuery=+status%3Adone+is%3Aopen+-label%3AStellar&pane=issue&itemId=68913749

Since Rafael is no longer here we will need to find the endpoint and how to integrate it in the FE to allow draft projects to be deleted. @kkatusic pick this one up whenever you have time.

@kkatusic
Copy link
Collaborator

@divine-comedian, I will take it, thx

@kkatusic
Copy link
Collaborator

kkatusic commented Sep 5, 2024

@laurenluz, cc @divine-comedian, we have problem with environment variable, after investigation @mhmdksh fixed. You can try to check change on staging domain.

@LatifatAbdullahi
Copy link

LatifatAbdullahi commented Sep 5, 2024

Test Update

When project status is "Draft", the "Delete Project" option is present - Pass

msr

When Project status is "Active" the "Delete Project" option is absent - Pass

fa

@kkatusic

Functional issue

However, I am unable to complete the delete process, as no action is taken after clicking on the "Delete Project" button

image

Latifat.Abdullahi._.Giveth.-.Google.Chrome.2024-09-05.17-20-57.mp4

UI issue

From the design above, the "Project name" should be bold as well.
The colour of the button should be Red not Pink
The button width should be 300px as per the design

nmmm

@kkatusic
Copy link
Collaborator

kkatusic commented Sep 6, 2024

@LatifatAbdullahi this function for delete is missing from code I'm investigating :)

@laurenluz
Copy link
Member Author

hey guys, this issue is on prod, but it's not working, see recording...
@kkatusic

Screen.Recording.2024-09-24.at.7.13.42.PM.mov

@divine-comedian
Copy link
Contributor

@kkatusic @LatifatAbdullahi I'm not sure this feature passed QA - so how did it end up in production?

@LatifatAbdullahi
Copy link

@divine-comedian

Last time this was tested, the delete action failed and it was still being investigated, that's why its still open.

@divine-comedian
Copy link
Contributor

OK but the code - which doesn't work - has been pushed to production, so now this is a problem that we have to focus on fixing and push a hotfix..

@divine-comedian divine-comedian removed P2 A normal to do item or feature. In Progress labels Sep 25, 2024
@kkatusic
Copy link
Collaborator

OK but the code - which doesn't work - has been pushed to production, so now this is a problem that we have to focus on fixing and push a hotfix..

no code pushed, just environment variable: NEXT_PUBLIC_DELETE_PROJECT_ENABLED let ask someone to setup it to false

@kkatusic
Copy link
Collaborator

@laurenluz, cc @divine-comedian, when you get time can you please check on production side is that option removed? thx

@laurenluz
Copy link
Member Author

remove from prod
image

@MohammadPCh MohammadPCh self-assigned this Sep 26, 2024
@divine-comedian
Copy link
Contributor

Thanks @kkatusic so we'll continue work on staging for the remaining bugs that @LatifatAbdullahi reported above
#1271 (comment)

@maryjaf
Copy link
Collaborator

maryjaf commented Oct 9, 2024

If you are busy with unicorn tasks , I can take this issue @LatifatAbdullahi

@maryjaf
Copy link
Collaborator

maryjaf commented Oct 9, 2024

Now the "delete" option is only shown in my project list, shouldn't this option also be shown in project page ? like as continu creation button a button there should be for delete?
@mosaeedi @divine-comedian

image

@divine-comedian
Copy link
Contributor

thanks @maryjaf - its a good suggestion but this feature isn't a high priority - I've made a new issue to follow up adding the button on the project details view, if all other tests pass we can call this issue done and follow up in the issue #4838

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

No branches or pull requests

8 participants