Skip to content

Implement AddParticipantsToEventPane and related functionality#150

Merged
rugged-bl merged 2 commits intomainfrom
add-participants-screen
Jan 24, 2026
Merged

Implement AddParticipantsToEventPane and related functionality#150
rugged-bl merged 2 commits intomainfrom
add-participants-screen

Conversation

@rugged-bl
Copy link
Contributor

@rugged-bl rugged-bl commented Jan 24, 2026

Note

Introduces an end-to-end flow for adding participants to an existing event and wires it into the app navigation and menu, with UI, domain use case, and tests, plus minor theme tweaks.

  • New AddParticipantsToEventPane (UI), AddParticipantsToEventViewModel, and nav AddParticipantsToEventPaneDestination
  • New domain use case AddParticipantsToCurrentEventUseCase and DI exposure via EventsComponent
  • Menu updated with "Add participants" action and navigation (MenuDialog, MenuViewModel, strings)
  • Integrated route into MainNavHost
  • Android instrumented tests added for the new flow and supporting test screens
  • Minor theming: set android:windowBackground with light/dark colors

Written by Cursor Bugbot for commit 6ca748e. This will update automatically on new commits. Configure here.

@rugged-bl rugged-bl requested a review from Copilot January 24, 2026 14:29
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Introduces an end-to-end flow for adding participants to an existing event, including UI, domain logic, navigation, and comprehensive testing support.

Changes:

  • Added new feature for adding participants to existing events with AddParticipantsToEventPane, AddParticipantsToEventViewModel, and domain use case
  • Integrated the feature into app navigation via MainNavHost and menu system with localized strings
  • Added Android instrumented tests to verify the complete flow and updated test screens for UI automation

Reviewed changes

Copilot reviewed 18 out of 18 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
MainNavHost.kt Registers the new add participants navigation module
MenuViewModel.kt Adds navigation handler for the add participants action
MenuDialogDestination.kt Wires the view model callback to the UI component
MenuDialog.kt Adds UI menu item for adding participants with icon and localized text
strings.xml (en, ru) Adds localized menu label for the add participants action
AddParticipantsToEventViewModel.kt Implements view model managing participant list state and confirmation logic
AddParticipantsToEventPaneUiModel.kt Defines UI state model with validation for confirming participants
AddParticipantsToEventPaneDestination.kt Sets up navigation destination and dependency injection
AddParticipantsToEventPane.kt Implements UI for adding multiple participants with dynamic form fields
AddParticipantsToCurrentEventUseCase.kt Domain use case handling participant insertion into current event
EventsComponent.kt Exposes the new use case via lazy dependency injection
themes.xml Sets window background color for light theme
colors.xml (light/dark) Defines window background colors for light and dark themes
MenuDialogScreen.kt Adds test helper method for navigating to add participants screen
AddParticipantsToEventScreen.kt New test screen class for UI automation of add participants flow
BasicInstrumentedTest.kt Adds comprehensive end-to-end test for the add participants feature

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@rugged-bl rugged-bl merged commit c6ba956 into main Jan 24, 2026
7 checks passed
@rugged-bl rugged-bl deleted the add-participants-screen branch January 24, 2026 14:32
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