Skip to content

refactor: add entity-specific Room DAOs alongside TreeTrackerDAO (#1235)#1303

Open
marsof02 wants to merge 6 commits into
Greenstand:masterfrom
marsof02:chore/split-treetracker-dao
Open

refactor: add entity-specific Room DAOs alongside TreeTrackerDAO (#1235)#1303
marsof02 wants to merge 6 commits into
Greenstand:masterfrom
marsof02:chore/split-treetracker-dao

Conversation

@marsof02

@marsof02 marsof02 commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Thank you for opening a Pull Request!

Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open a GitHub issue as a bug/feature request before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests are added/updated (if necessary)
  • Ensure the linter passes (./codeAnalysis to automatically apply formatting/linting)
  • Appropriate docs were updated (if necessary)
  • This change has no user-visible / UI effect (refactor, docs, CI, etc.), so screenshots are not applicable.

Part of #1235 🦕

Summary

First PR in a stack to split TreeTrackerDAO into entity-specific DAOs.

  • Adds 7 DAO interfaces under database/dao/
  • Wires them in AppDatabase and RoomModule
  • Keeps TreeTrackerDAO for now — caller migration in follow-up PRs
    Plan: docs/tasks/task_22/README.md

Out of scope

  • Migrating repos, uploaders, use cases, and view models
  • Removing TreeTrackerDAO
  • Splitting TreeTrackerDaoTest
1235_CI.mp4

@cynthiabondi

Copy link
Copy Markdown
Contributor

Great work @marsof02 !

@github-actions

github-actions Bot commented Jun 27, 2026

Copy link
Copy Markdown

✅ Thanks @marsof02 — required media detected. Nothing more needed here!

@marsof02 marsof02 force-pushed the chore/split-treetracker-dao branch from b7753e9 to e197a3d Compare June 30, 2026 01:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Ready for Review

Development

Successfully merging this pull request may close these issues.

3 participants