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

Add moderated communities to drawer #1063

Merged
merged 1 commit into from
Jan 17, 2024
Merged

Conversation

hjiangsu
Copy link
Member

@hjiangsu hjiangsu commented Jan 17, 2024

Pull Request Description

This PR takes the initial steps for adding moderation capabilities to Thunder.

  • I've refactored AccountBloc to be a bit more modular to hopefully reduce unnecessary API calls.
  • AccountBloc now stores a list of moderated communities for the current user. This should allow us to use this information across the whole application for moderation actions
  • I've added an additional section to the drawer which contains moderated communities for the current user.

This is step 1 of moderator actions. Once this is merged in, the next steps can be taken:

  • Create a bloc to handle moderation related actions
  • Add new actions for moderated communities to handle reporting, deleting, etc.

Issue Being Fixed

Issue Number: #1007

Screenshots / Recordings

With favourited communities
simulator_screenshot_FBFCD2C1-DBAC-4962-B25F-73A62360BD7B simulator_screenshot_2209769F-57F5-470F-90D8-108F9C849C8C

Checklist

  • Did you update CHANGELOG.md?
  • Did you use localized strings where applicable?
  • Did you add semanticLabels where applicable for accessibility?

Copy link
Member

@micahmo micahmo left a comment

Choose a reason for hiding this comment

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

LGTM, very nice!

@hjiangsu hjiangsu merged commit 226a382 into develop Jan 17, 2024
1 check passed
@hjiangsu hjiangsu deleted the feature/moderator-sidebar branch January 17, 2024 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants