Feature Request: Remove the old home activity/fragment and other related files [Blocked]

[theMr17]

Is your feature request related to a problem? Please describe.

Issue #5344 tracks the implementation of the new Classroom List Screen, which will replace the existing Home Screen. Once the new screen is stable, all unused code related to the old screen, including the ENABLE_MULTIPLE_CLASSROOMS feature flag, should be removed.

Describe the solution you'd like

Here is a brief overview of the steps:

• Remove HomeActivity, HomeFragment & its associated presenters along with HomeViewModel.
• Migrate the following files from the app/home to the app/classroom package.
  • app/home/HomeItemViewModel
  • app/home/RouteToExplorationListener
  • app/home/RouteToRecentlyPlayedListener
  • app/home/RouteToTopicListener
  • app/home/RouteToTopicPlayStoryListener
• Migrate the following subpackages from app/home to app/classroom package. (Note: corresponding subpackages already exist in app/classroom. Move all files from app/home to those subpackages):
  • app/home/classroomlist/..
  • app/home/promotedlist/..
  • app/home/recentlyplayed/..
  • app/home/topiclist/..
• Delete the app/home package after the migration.
• Remove the getTopicList function from TopicListController and all related functions (such as createTopicList, loadTopicListFromJson, createEphemeralTopicSummary, and so on). Note: the Classroom List Screen now uses the new ClassroomController for the getTopicList function.
• Remove the gating logic for HomeActivity and ClassroomListActivity from the following activities to directly open ClassroomListActivity:
  • PinPasswordActivity
  • ProfileChooserFragment
  • NavigationDrawerFragment
  • MyDownloadsActivity
• Remove the ENABLE_MULTIPLE_CLASSROOMS feature flag.

Additional context

Ensure that all corresponding test files are removed or updated accordingly for each of the above changes.

Note: This issue is blocked until the Multiple Classrooms feature is stable and will be ready to work on once approval is received from the TLs.

[seanlip]

@theMr17 Could you please update the list of steps for this issue? Thanks.

[theMr17]

@seanlip I realized that I forgot to add the steps after including the TODO earlier. The steps have now been added.