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

Comments on posts are not being deleted #1

Open
sophiajwitt opened this issue Sep 7, 2023 · 0 comments · May be fixed by #4, #5, #8, #16 or #22
Open

Comments on posts are not being deleted #1

sophiajwitt opened this issue Sep 7, 2023 · 0 comments · May be fixed by #4, #5, #8, #16 or #22

Comments

@sophiajwitt
Copy link
Contributor

sophiajwitt commented Sep 7, 2023

As an admin, when I delete a comment on a post, the interface tells me it has been deleted, however when I check to see if the comment is gone it's still there.

Here is a video representation of the bug

@Sweetsuro Sweetsuro linked a pull request Sep 20, 2023 that will close this issue
This was referenced Sep 20, 2023
@yuvanshu yuvanshu linked a pull request Sep 20, 2023 that will close this issue
@SerenaYaoo SerenaYaoo linked a pull request Sep 20, 2023 that will close this issue
@lwunder lwunder linked a pull request Sep 20, 2023 that will close this issue
@BojunLi03 BojunLi03 linked a pull request Sep 20, 2023 that will close this issue
hanahryu pushed a commit to hanahryu/CMU-313-fall23-sw-archaeology-recitation that referenced this issue Sep 20, 2023
@yliang412 yliang412 linked a pull request Sep 20, 2023 that will close this issue
@YuheMa702 YuheMa702 linked a pull request Sep 20, 2023 that will close this issue
@aaronmarm aaronmarm linked a pull request Sep 20, 2023 that will close this issue
@juhyep juhyep linked a pull request Sep 22, 2023 that will close this issue
@TBennur TBennur linked a pull request Sep 22, 2023 that will close this issue
gonzalezjo added a commit to gonzalezjo/CMU-313-fall23-sw-archaeology-recitation that referenced this issue Sep 22, 2023
Comments previously were not being deleted because after deletion in the DB, we'd return without actually committing the changes. Now, we commit the changes, and so it should actually work.

Resolves CMU-313#1
@gonzalezjo gonzalezjo linked a pull request Sep 22, 2023 that will close this issue
@arustogi arustogi mentioned this issue Sep 22, 2023
@hankxu1212 hankxu1212 linked a pull request Sep 22, 2023 that will close this issue
@jonathanho168 jonathanho168 linked a pull request Sep 22, 2023 that will close this issue
@jpurista jpurista linked a pull request Sep 22, 2023 that will close this issue
@kc12341 kc12341 linked a pull request Sep 22, 2023 that will close this issue
@kayliny02 kayliny02 linked a pull request Sep 22, 2023 that will close this issue
@connormaas-cmu connormaas-cmu linked a pull request Sep 22, 2023 that will close this issue
@ryan-huan ryan-huan linked a pull request Sep 22, 2023 that will close this issue
@joycehua joycehua linked a pull request Sep 22, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment