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

Fix Issue : 55 #62

Merged
merged 8 commits into from
Oct 12, 2021
Merged

Fix Issue : 55 #62

merged 8 commits into from
Oct 12, 2021

Conversation

rooparsh
Copy link
Contributor

@rooparsh rooparsh commented Oct 7, 2021

This PR restructures the project hierarchy and make the code more modular
Addresses the issue Link

@acmbot
Copy link

acmbot bot commented Oct 7, 2021

Thanks @rooparsh, for opening the pull request! 🙌
One of our team-mates will review the pull request soon. ✅

@DarthBenro008
Copy link
Member

@rooparsh , could you kindly share your views on this comment?

#55 (comment)

@rooparsh
Copy link
Contributor Author

rooparsh commented Oct 8, 2021

@DarthBenro008 if you take a pull of this branch and compare with main branch, you will understand what I am saying

@gaganmalvi
Copy link
Member

I'm sorry I was AFK for the past few days, I will check asap.

@gaganmalvi
Copy link
Member

Can you fix the merge conflicts?
I'll go ahead with the merge

# Conflicts:
#	app/src/main/java/org/acmvit/gitpositive/MainActivity.kt
#	app/src/main/java/org/acmvit/gitpositive/ui/home/HomeScreen.kt
#	app/src/main/res/layout/activity_main.xml
@rooparsh
Copy link
Contributor Author

@gaganmalvi Conflicts resolved

@rishuyadav
Copy link
Member

Looks good and organized, thanks for contributing

@rishuyadav rishuyadav merged commit 98b9851 into ACM-VIT:main Oct 12, 2021
@acmbot
Copy link

acmbot bot commented Oct 12, 2021

Congratulations @rooparsh, your pull request is merged! 🎉
Thanks for your contributions. 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants