mentorship-android icon indicating copy to clipboard operation
mentorship-android copied to clipboard

feat: New Window for Searching Members

Open bosankus opened this issue 3 years ago • 6 comments

Description

Created a separate fragment for searching members i.e SearchMembersFragment. This solves the below mentioned issue, and is implemented as per the discussion on issue #719.

Fixes #732

Type of Change:

  • Code
  • User Interface

Code/Quality Assurance Only

  • New feature (non-breaking change which adds functionality pre-approved by mentors)

How Has This Been Tested?

Checklist:

  • [x] My PR follows the style guidelines of this project
  • [x] I have performed a self-review of my own code or materials
  • [x] I have commented my code or provided relevant documentation, particularly in hard-to-understand areas

Code/Quality Assurance Only

  • [x] My changes generate no new warnings
  • [ ] I have added tests that prove my fix is effective or that my feature works - Espresso testing added but needs to be reviewed
  • [x] New and existing unit tests pass locally with my changes

bosankus avatar Nov 12 '21 01:11 bosankus

@isabelcosta @epicadk facing one issue in this PR. That is the added espresso testing. Though the feature works fine. It will be helpful if any of you can help me to review the testing part. Also I will be creating a new issue for the back press issue which is getting faced in this new screen. 'Cancel' button removes the fragment and works smooth. Kindly share any ideas regarding this.

bosankus avatar Nov 12 '21 01:11 bosankus

@isabelcosta @epicadk facing one issue in this PR. That is the added espresso testing. Though the feature works fine. It will be helpful if any of you can help me to review the testing part. Also I will be creating a new issue for the back press issue which is getting faced in this new screen. 'Cancel' button removes the fragment and works smooth. Kindly share any ideas regarding this.

Yeah for some reason I couldn't get the tests to work either. @isabelcosta do you think we could create an issue for this and perhaps only manually test it now?

epicadk avatar Nov 12 '21 04:11 epicadk

@isabelcosta can you check kindly?

bosankus avatar Nov 15 '21 18:11 bosankus

@bosankus thank you for your inputs, do create the issue if you have time as you have got a better understanding of it, if not that's alright as well. We can proceed with manual testing of the feature!

vj-codes avatar Nov 18 '21 21:11 vj-codes

@vj-codes Sure, I will create the issue.

bosankus avatar Nov 19 '21 01:11 bosankus

This feature will be improved due to implementation of paging3 in MembersFragment from issue #1166. Closing this issue due to duplicate.

bosankus avatar Dec 03 '21 04:12 bosankus