search

fossasia / open-event-organizer-android

Open Event Mobile App for Organizers and Entry Managers https://play.google.com/store/apps/details?id=com.eventyay.organizer
Apache License 2.0
1.79k stars 490 forks source link

Move to Architecture Components and MVVM #562

Open iamareebjamal opened 6 years ago

iamareebjamal commented 6 years ago

Reimplement MVP with MVVM through

iamareebjamal commented 6 years ago

This will be done eventually and is already in place, so closing

sridharjajoo commented 6 years ago

@iamareebjamal Can you open this issue because Mario has said that as it hasn't been implemented fully so it needs to be opened.

mariobehling commented 6 years ago

@sridharjajoo @Masquerade0097 @mishuvs Please add sub-issues.

dr0pdb commented 6 years ago

@sridharjajoo @mishuvs @Masquerade0097 Let's focus on converting the list fragments to MVVM first since it will automatically fix #1079. The details and other fragments are working well in any case. So first the list fragments.

sridharjajoo commented 6 years ago

@srv-twry Sure. Will create new issues for it and work upon them.

ShridharGoel commented 5 years ago

I would like to work on the remaining Presenters which need to be migrated to ViewModels.

iamareebjamal commented 5 years ago

Please open issue you are currently working on. We don't want 10000 sub issues unless a PR is ready

ShridharGoel commented 5 years ago

Okay, sorry for that.