scrapbook icon indicating copy to clipboard operation
scrapbook copied to clipboard

feat: add option to delete posts from web UI

Open cskartikey opened this issue 1 year ago • 6 comments

Should close #746

Todo:

  • [x] Figure out a way to check if authenticated user is the owner of the post.

cskartikey avatar Jun 02 '24 15:06 cskartikey

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

Name Status Preview Comments Updated (UTC)
scrapbook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 31, 2024 5:00am

vercel[bot] avatar Jun 02 '24 15:06 vercel[bot]

📝 Changed routes:

Commit 7e23d597d1b901dde724add87bcece45d7f6ffb6 (https://scrapbook-bt9gnexbi.hackclub.dev).

lachlanjc avatar Jun 03 '24 10:06 lachlanjc

please add a toast that will indicate to the user the status of the operation (success, error, loading)

JosiasAurel avatar Jun 05 '24 11:06 JosiasAurel

i should not have the option to delete a post that i did not create from the web interface

I would say one should have the option. I mean, it would be a lot more work to go back to slack to delete it (considering it was posted via slack)

NB: I haven't checked how the code works, I left this comment based on what was said in the earlier comment. Correct me if I said something wrong :)

Ranger-NF avatar Jul 10 '24 12:07 Ranger-NF

i should not have the option to delete a post that i did not create from the web interface

I would say one should have the option. I mean, it would be a lot more work to go back to slack to delete it (considering it was posted via slack)

NB: I haven't checked how the code works, I left this comment based on what was said in the earlier comment. Correct me if I said something wrong :)

I think there's a misunderstanding here. What I meant is that I should not have the option to delete a post that I did not create using my slack account or my web scrapbook account.

JosiasAurel avatar Jul 10 '24 12:07 JosiasAurel

Aah... My bad. I agree with you then

Ranger-NF avatar Jul 10 '24 14:07 Ranger-NF