Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Adding ability to remove and restore comments. #1182#1323

Merged
dessalines merged 6 commits into
mainfrom
comment_removing
Jan 18, 2024
Merged

Adding ability to remove and restore comments. #1182#1323
dessalines merged 6 commits into
mainfrom
comment_removing

Conversation

@dessalines

@dessalines dessalines commented Jan 9, 2024

Copy link
Copy Markdown
Member

This adds the ability to remove and restore comments.

It also adds the correct hierarchical logic for the canMod function copied from lemmy-ui, which requires a list of admins and mods.

dessalines
dessalines commented Jan 9, 2024
View reviewed changes

@dessalines dessalines left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I plan on adding more mod-abilities to jerboa in the coming weeks, but this is a good start.

Comment thread app/src/main/java/com/jerboa/Utils.kt
Comment thread app/src/main/java/com/jerboa/model/RemoveViewModel.kt
Comment thread app/src/main/java/com/jerboa/ui/components/comment/CommentNode.kt Outdated
Comment thread app/src/main/java/com/jerboa/ui/components/comment/CommentNode.kt Outdated
Comment thread app/src/main/java/com/jerboa/ui/components/comment/CommentOptionsDropdown.kt
Comment thread app/src/main/java/com/jerboa/ui/components/person/PersonProfileActivity.kt
Comment thread app/src/main/java/com/jerboa/ui/components/remove/comment/CommentRemoveActivity.kt
Comment thread app/src/main/java/com/jerboa/ui/components/remove/comment/CommentRemoveActivity.kt
MV-GH
MV-GH requested changes Jan 10, 2024
View reviewed changes

@MV-GH MV-GH left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can't test the functionality, not a mod somewhere but I'll take your word on it.

Comment thread app/src/main/java/com/jerboa/model/RemoveViewModel.kt
Comment thread app/src/main/java/com/jerboa/ui/components/comment/CommentNode.kt Outdated
Comment thread app/src/main/java/com/jerboa/ui/components/comment/CommentNode.kt Outdated
Comment thread app/src/main/java/com/jerboa/ui/components/comment/CommentNode.kt Outdated
Comment thread app/src/main/java/com/jerboa/ui/components/comment/CommentOptionsDropdown.kt
Comment thread app/src/main/java/com/jerboa/ui/components/remove/comment/CommentRemoveActivity.kt
@dessalines dessalines mentioned this pull request Jan 11, 2024
Closed
4 tasks
@dessalines

dessalines commented Jan 11, 2024

Copy link
Copy Markdown
Member Author

I've been using voyager.lemmy.ml to test. You can use my test admin account there:

username: dess_tester_3

pass: dess_tester_3dess_tester_3

@dessalines dessalines mentioned this pull request Jan 11, 2024
Merged
@dessalines dessalines requested a review from MV-GH January 11, 2024 23:50
@dessalines dessalines merged commit 47ee753 into main Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MV-GH MV-GH MV-GH approved these changes
@twizmwazin twizmwazin Awaiting requested review from twizmwazin

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dessalines @MV-GH