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

Added margin in Group details activity #1206

Merged
merged 1 commit into from
Dec 19, 2019

Conversation

PiyushSharma99
Copy link

@PiyushSharma99 PiyushSharma99 commented Dec 4, 2019

Fixes #1192

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

  • Apply the MifosStyle.xml style template to your code in Android Studio.

  • 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.

@PiyushSharma99
Copy link
Author

@santoshconflux @luckyman20 @jddeep @harshinde07 Please review and approve. Thanks.

@luckyman20
Copy link
Collaborator

@PiyushSharma99 Attach screenshot too.

@PiyushSharma99
Copy link
Author

@luckyman20 Added screenshot. Thanks!

Copy link

@jddeep jddeep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me @PiyushSharma99 👍

@PiyushSharma99
Copy link
Author

@luckyman20 Please review and approve. @jddeep Could you please approve my submission on the gci portal?

@luckyman20
Copy link
Collaborator

luckyman20 commented Dec 6, 2019

Push the changes to development branch not to master branch.

Also, change your commit tile to fix: add margin in Group details activity.

@PiyushSharma99
Copy link
Author

@luckyman20 How can I change this now? I mean change the base branch where I want to push the changes. Do i open a new PR?

@PiyushSharma99
Copy link
Author

@luckyman20 There is no development branch!

@luckyman20
Copy link
Collaborator

@PiyushSharma99 Sorry, my bad. This project doesn't have that. Just fix up the commit title and you would be good to go.

@PiyushSharma99
Copy link
Author

Done 👍 @luckyman20

@luckyman20 luckyman20 merged commit 4490f18 into openMF:master Dec 19, 2019
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.

Add Margin to GroupActivity [GCI]
3 participants