update assigned moderators list behavior#286
Merged
steryokhin merged 4 commits intodevelopfrom Feb 15, 2026
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request enhances the moderators list behavior in the Feeding Point Details screen by introducing explicit expand/collapse functionality with a chevron icon and a "Show All" action with a count indicator. Previously, the moderators section had limited and unclear interaction patterns.
Changes:
- Added localized "Show All (count)" action button to reveal all moderators when the list exceeds 5 items
- Introduced chevron toggle for expanding/collapsing the entire moderators section
- Centralized moderators display logic in the ViewModel with new state management for expansion and "Show All" interactions
- Modified Model to return full moderators list without hard-coded limits
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| Strings.swift | Added showAll(_:) localization function with count parameter |
| FeedingPointDetailsModel.swift | Removed 10-item limit; now returns complete moderators list |
| FeedingPointDetailsViewController.swift | Implemented "Show All" button and chevron expand/collapse UI |
| FeedingPointDetailsViewMapper.swift | Added isExpanded parameter to moderators mapping |
| FeedingPointDetailsContract.swift | Added tapToggleModeratorsVisibility event |
| FeedingPointDetailsViewModel.swift | Implemented expand/collapse state management with "Show All" logic |
| sheet.csv | Added localization entries for new strings |
| en.lproj/Localizable.strings | Added English translation and updated "About Us" to "About" |
| ka.lproj/Localizable.strings | Added Georgian translation for "Show All" |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
...src/Flows/Main/Modules/Home/FeedingPointDetails/View/FeedingPointDetailsViewController.swift
Outdated
Show resolved
Hide resolved
animeal/src/Flows/Main/Modules/Home/FeedingPointDetails/FeedingPointDetailsViewModel.swift
Show resolved
Hide resolved
animeal/src/Flows/Main/Modules/Home/FeedingPointDetails/FeedingPointDetailsViewModel.swift
Outdated
Show resolved
Hide resolved
steryokhin
reviewed
Feb 10, 2026
...src/Flows/Main/Modules/Home/FeedingPointDetails/View/FeedingPointDetailsViewController.swift
Outdated
Show resolved
Hide resolved
animeal/src/Flows/Main/Modules/Home/FeedingPointDetails/FeedingPointDetailsViewModel.swift
Outdated
Show resolved
Hide resolved
Collaborator
|
Forgot to submit review :( |
steryokhin
approved these changes
Feb 15, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request overview
Assigned Moderators: Show All + Expand / Collapse
Problem
Summary
Changes
L10n.Action.showAll(_:)localizationisExpandedto moderators mappingtapToggleModeratorsVisibilityeventHow it works
How to test
Moderator / Admin
Volunteer