-
Notifications
You must be signed in to change notification settings - Fork 60
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
Ali Shobeyri #53
Open
sasssass
wants to merge
73
commits into
daresaydigital:master
Choose a base branch
from
sasssass:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Ali Shobeyri #53
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
description : 1.create MotherFragment.kt and it's functions 2.add json lottie for loading 3.add colors
description : 1.create MotherViewModel.kt and it's functions 2.add cast mapper 3.add Domain
description : 1.create a mapper for converting dto to domain 2.create a safeApi function for network
description : 1.add basic tools for navigation system
description : 1.add 2 further features navigation files
description : 1.add functionality for navigate between 2 modules (home and movie)
description : 1.change policy of navigation system 2.refactor MainActivity.kt with dataBinding and add navigation to it
description : 1.add movie and home feature 2.separate movie module into separate modules 3.add gradle dependencies
description : 1.fix gradle 2.add dto model for discover movie api
description : 1.add discover movie model for domain 2.add Mapper for data dto of discover movie
description : 1.add repository interface to domain
description : 1.change safeApi.kt and remove flow from it 2.add implementation of MovieRepository 3.add movie datasource
description : 1.add DI for data layer
description : 1.add Mother Usecase for base of other use cases
description : 1.add proper usecase for discovering movies
description : 1.add global query param as api_key
description : 1.add base test to base data 2.add test for data layer of movie
description : 1.add test for mapper
description : 1.add test for safeApi
description : 1.init the module
description : 1.create Homefragment
description : 1.fix ui of home fragment 2.add navigation system for home module
description : 1.add click extension for dataBinding
description : 1.connect home module to movie module
description : 1.add DI 2.add viewModel and proper logic for whole steps 3.add layout 4.connect to navigation system
description : 1.add general adapter for lists
description : 1.implement the whole steps to getting the movies from API
description : 1.add a button for navigating to offline page
description : 1.change policy of getting data because now we have offline data too
description : 1.add viewModel for detail screen
description : 1.change deleting function
description : 1.write proper methods for adding and deleting data from db
description : 1.chang movie detail fragment for deleting and saving movies 2.add active ext for views 3.add toast ext
description : 1.add di for room
description : 1.fix crash when you want to delete a favorite item
description : 1.complete tests for movie module 2.fix bug and crashes
description : 1.add comments
description : 1.add comments
description : 1.add comments
description : 1.add comments
description : 1.add comments
description : 1.add empty state for list
description : 1.fix adult tag
description : 1.add diagram
description : 1.reformat and organize codes
description : 1.add shared domain 2.refactor movie module and add top rated service
description : 1.change the UI, current one is not good 2.optimise base ui
description : 1.change margins in fragment detail
description : 1.optimise list
description : 1.optimise UI 2.change tests depending on new UI
description : 1.change UI
description : 1.change UI
description : 1.change navigation policy
description : 1.improve UI
description : 1.improve UI
description : 1.add Alert Dialog 2.show pop when an error happens
description : 1.refactor gradle
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Ali Shobeyri
Android