Skip to content

:APP Part_4 Folder rename#67

Open
IntFxZen wants to merge 9 commits intoembedded-dev-research:mainfrom
IntFxZen:part4-folder-rename
Open

:APP Part_4 Folder rename#67
IntFxZen wants to merge 9 commits intoembedded-dev-research:mainfrom
IntFxZen:part4-folder-rename

Conversation

@IntFxZen
Copy link
Copy Markdown
Contributor

Overview

This PR introduces the folder rename flow in the UI layer and wires it end-to-end through the existing architecture.
The change keeps the scope small and focused: add rename behavior without expanding into unrelated note/folder features.

Changes

  • Domain UseCases Wiring: Added UpdateFolderUseCase and GetFolderUseCase into app-level DI and NotesUseCases aggregation.
  • UI Contract: Extended NotesUiEvent with RenameDirectory to formalize rename intent between UI and ViewModel.
  • ViewModel Logic: Implemented rename handling in NotesViewModel with input normalization and guardrails (blank names and all folder protection).
  • UI Interaction: Updated DirectoriesScreen long-press flow to support directory actions and added rename dialog with prefilled current name.
  • App Wiring: Connected rename callback from NotesApp to NotesViewModel event dispatch.

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.

1 participant