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.

Sign up for GitHub

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

Jump to bottom

Search refactor [WIP] #143

Merged
merged 40 commits into from
Jul 1, 2015
Merged

Search refactor [WIP] #143

merged 40 commits into from
Jul 1, 2015

Conversation

pierr
Copy link
Contributor

@pierr pierr commented Jun 26, 2015

store

  • Remove searchstore and querystore
  • Add an advancedSearch store and a quickSearchStore

Search

  • Focus.search.builtInStore.{acvancedSearchStore, quickSearchStore}
  • Focus.search.actionBuilder(conf)=>{searchAction, updateQuery, UpdateScope}

@pierr
Copy link
Contributor Author

pierr commented Jun 26, 2015

Action content needs to be implemented

@pierr
Copy link
Contributor Author

pierr commented Jun 29, 2015

See with @Bernardstanislas :

  • Add a store in the config to read the state
  • Add a Interface store

@pierr
Copy link
Contributor Author

pierr commented Jun 30, 2015

Fix #105

@Bernardstanislas
Copy link
Contributor

pierr added a commit that referenced this pull request Jul 1, 2015
@pierr
Merge pull request #143 from KleeGroup/search-refactor
4abbc00 
Search refactor [WIP]
@pierr pierr merged commit 4abbc00 into master Jul 1, 2015
@pierr pierr deleted the search-refactor branch July 1, 2015 17:49
@pierr
Copy link
Contributor Author

pierr commented Jul 1, 2015

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@pierr @Bernardstanislas