Skip to content

Search functionality and few bug fixes#156

Merged
opticod merged 9 commits intoBuildmLearn:bug-fixesfrom
tejavojjala:master
Sep 3, 2016
Merged

Search functionality and few bug fixes#156
opticod merged 9 commits intoBuildmLearn:bug-fixesfrom
tejavojjala:master

Conversation

@tejavojjala
Copy link
Copy Markdown
Contributor

@tejavojjala tejavojjala commented Aug 30, 2016

Bugs fixed are one due to objects.equals and other is becoz of adding header view after the adapter is set in mainfragment of matchtemplate it's making the app crash.
Apart from these after i added search functionality a few bugs came in but i resolved those now.

Teja Vojjala added 9 commits August 26, 2016 21:19
# The first commit's message is:

fixed app crash in phones with api level<19

# This is the 2nd commit message:

fixed app crash in phones with api level<19
setting adapter before adding views results in app
crash in few versions of android
@opticod
Copy link
Copy Markdown
Collaborator

opticod commented Aug 31, 2016

Can you mention the list of bugs fixed by this PR? @iit2014086

@opticod opticod self-assigned this Sep 3, 2016
@opticod opticod added the testing label Sep 3, 2016
@opticod
Copy link
Copy Markdown
Collaborator

opticod commented Sep 3, 2016

🎉 LGTM merging.

Small note:

  1. Try to squash your commits.
  2. The search functionality can further be improved by
  • making the search case in-sensitive
  • matching all strings of type : %text_to_found% presently it only matches text_to_found%

@opticod opticod merged commit e5415eb into BuildmLearn:bug-fixes Sep 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants