Skip to content

Conversation

@vikasrohit
Copy link

-- Applied fix from winner of the challenge. This fix has least intrusive changes with only two files being changed and only 3 lines of addition overall. Other change which I committed in separate branch (feature/add_team_member_scroll) works fine but has changes in more 6-7 files and requires functional component to be stateful. So, we are going to accept this fix as final solution.

…ing on Add button

-- Applied fix from winner of the challenge. This fix has least intrusive changes with only two files being changed and only 3 lines of addition overall. Other change which I committed in separate branch (feature/add_team_member_scroll) works fine but has changes in more 6-7 files and requires functional component to be stateful. So, we are going to accept this fix as final solution.
@vikasrohit vikasrohit merged commit 45221d8 into dev Nov 15, 2016
@vikasrohit vikasrohit deleted the feature/add_team_memeber_scroll_community_fix branch December 5, 2016 06:26
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.

3 participants