Skip to content

Refactor: kind of MVC -> MVVM#43

Merged
JeelDobariya38 merged 15 commits intomainfrom
jeel-dev
Oct 10, 2025
Merged

Refactor: kind of MVC -> MVVM#43
JeelDobariya38 merged 15 commits intomainfrom
jeel-dev

Conversation

@JeelDobariya38
Copy link
Member

@JeelDobariya38 JeelDobariya38 commented Oct 7, 2025

[FIRST STEP TO BIGGEST MIGRATION]

Changes Made

  • Add Koin for dependency injection.
  • Make view model for activities.

Note

This PR contains code that seems bad or not a good practice.. But that is only to helping the migration towards jet-pack compose. #42. and this code is temporary..

@JeelDobariya38 JeelDobariya38 self-assigned this Oct 7, 2025
@JeelDobariya38 JeelDobariya38 added the refactor refactor the code label Oct 7, 2025
@JeelDobariya38 JeelDobariya38 merged commit 8652c3b into main Oct 10, 2025
@hexCode63
Copy link
Member

Have you completely changed my autofill code?

@JeelDobariya38
Copy link
Member Author

Have you completely changed my autofill code?

I haven't even touched it, I am not planning to work on it utill, i am done with 1 tier of migration... #42..

But I will surely change it, so to speak "make improvements on top of your code" ... which serve as foundation.

I don't know autofill in depth... but I know a repository, where I can look for for inspiration... yogeshpaliyal/KeyPass

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor refactor the code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants