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

[Similar Projects] MBL-2164: Add a Use Case object with stubs #2311

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

stevestreza-ksr
Copy link
Contributor

📲 What

This PR adds a stub implementation of a Use Case object for Similar Projects. It works except for loading data which will come later. And also we will actually use this in future work as well.

🤔 Why

ProjectPageViewModel is already big enough, we don't need to add more responsibilities to it here :)

This is also to aid in parallelization of work.

🛠 How

After planning out the responsibilities that need to be handled in a view model, I built a few stubs of what those types will look like. I also added unit tests and integrated this into the ProjectPageViewModel.

👀 See

Spike

✅ Acceptance criteria

Tests should pass

@stevestreza-ksr stevestreza-ksr changed the title [Similar Projects] Add a Use Case object with stubs [Similar Projects] MBL-2164: Add a Use Case object with stubs Mar 5, 2025
Copy link
Contributor

@scottkicks scottkicks left a comment

Choose a reason for hiding this comment

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

Love seeing more and more inline documentation!

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