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

New HomeScreen Design #13

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

New HomeScreen Design #13

wants to merge 7 commits into from

Conversation

fatahrez
Copy link
Collaborator

@fatahrez fatahrez commented Jun 7, 2022

Description

  • This pull request refurbishes the code in the HomeScreen
  • Added Search bar
  • Added Category for different drinks
  • Added Filter for glasses

How should this be manually tested?

  • Clone the App, Run it in Android Studio and install it on your phone

Type of change

Please select the relevant option

  • Bug fix(a non-breaking change that fixes an issue)
  • New feature(a non-breaking change that adds functionality)
  • Breaking change(fix a feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes

  • End to end test
  • Integration test
  • [] unit test

Checklist:

  • My code follows the style guidelines
  • I have Linted my code prior to submission
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • [] I have added tests
  • [] New and existing tests pass locally with my changes

@MamboBryan
Copy link
Contributor

@fatahrez This is a PR and a half, awesome work man

@fatahrez
Copy link
Collaborator Author

fatahrez commented Jun 8, 2022

Thanks @MamboBryan

@daniel-waiguru
Copy link

@fatahrez This is a PR and a half, awesome work man

i tell you, good job @fatahrez

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.

None yet

3 participants