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 viewpager2 in the favorites fragment, with 2 tabs #27

Merged

Conversation

krishnachaitanya0107
Copy link
Contributor

@krishnachaitanya0107 krishnachaitanya0107 commented Oct 5, 2021

Added viewpager2 in the favorites fragment, with 2 tabs .

Copy link
Owner

@justdvnsh justdvnsh left a comment

Choose a reason for hiding this comment

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

Hi @krishnachaitanya0107 I have reviewed this PR. Please make the required changes. Thanks.

@justdvnsh justdvnsh added the changes-requested attached when a change is requested in the PR. label Oct 5, 2021
@krishnachaitanya0107
Copy link
Contributor Author

@justdvnsh thank you for going through the code and suggesting improvements to the code , i have implemented these changes , please go through it and do let me know if there are any more things that can be improved . I will be happy to help .

@justdvnsh
Copy link
Owner

Looks Good to me @krishnachaitanya0107 Thank you so much for your contribution. If you'd like, please join the slack channel for further discussions (You can find the link in the readme). Thanks again.

@justdvnsh justdvnsh added hacktoberfest-accepted Attached when pr is ready to merge and can be counted for hacktoberfest. ready-to-merge attached when a pr is ready to be merged UI-change Label to put if there is any UI change and removed changes-requested attached when a change is requested in the PR. labels Oct 5, 2021
@justdvnsh justdvnsh merged commit 3dee1b9 into justdvnsh:develop Oct 5, 2021
@krishnachaitanya0107
Copy link
Contributor Author

Thank you for accepting the pull request , it really means a lot to me . I would definitely love to join the slack channel . It felt really great working on this project and i would love to contribute and work on more issues on this project . Thank you for this wondeful opportunity .

@justdvnsh
Copy link
Owner

@krishnachaitanya0107 I am personally so happy to have contributors like you. best of luck for your open source journey. I am happy this project could have helped.

@justdvnsh
Copy link
Owner

@krishnachaitanya0107 also, I am going to launch another app in a few days. An open source app to download wallpapers for your phone. Without any ads. I would be happy if you'd contribute there as well. Moreover, this project is in heavy development, so new issues are being added everyday. Please contribute more to the project if you'd like.

@krishnachaitanya0107
Copy link
Contributor Author

I will definitely check out that project and i will contribute to both the projects by resolving issues which i feel comfortable with . Thank you for helping me in my open source journey and for providing valuable feedback about my code to help improve my coding skills as well .

@krishnachaitanya0107 krishnachaitanya0107 deleted the favourites_viewpager_feature branch October 12, 2021 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Attached when pr is ready to merge and can be counted for hacktoberfest. ready-to-merge attached when a pr is ready to be merged UI-change Label to put if there is any UI change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants