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

Home Fragment, Transfer Fragment, Profile Fragment and Account Fragment #166

Closed
wants to merge 1 commit into from

Conversation

garvit984
Copy link
Collaborator

@garvit984 garvit984 commented Mar 25, 2020

Fixes #102 #103 #105 #107
Basic UI design of Home fragment, Profile Fragment and Account Fragment designed during GCI is implemented.
Please Add Screenshots If there are any UI changes.
ezgif-3-24ed7a2ee3e9
Screenshot_2020-04-17-13-39-49-286_org mifos mobile cn

Please make sure these boxes are checked before submitting your pull request - thanks!

  • Run the unit tests with ./gradlew check to make sure you didn't break anything

  • If you have multiple commits please combine them into one commit by squashing them.

@garvit984
Copy link
Collaborator Author

@abhijitramesh @therajanmaurya @edcable Please review it

@garvit984 garvit984 changed the title Home Fragment, Profile Fragment and Account Fragment Home Fragment, Transfer Fragment, Profile Fragment and Account Fragment Apr 17, 2020
@rchtgpt
Copy link
Member

rchtgpt commented May 16, 2020

hello @garvit984, as much as I appreciate your humongous contribution, I feel very sorry to tell you that the Issues that you have fixed, have already been fixed in #115, #112, #117, #118 and await a review from the senior project maintainers. It is always advised to not create a duplicate PR hence until any already existing PR for these issues are closed, we cannot consider this PR and so I will have to close it for now.

@rchtgpt rchtgpt closed this May 16, 2020
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.

2 participants