Search API, UI and their integration #113

Closed
wants to merge 4 commits into
from

Conversation

Projects
None yet
2 participants
@D-Ne0
Contributor

D-Ne0 commented May 30, 2014

This pull doesn't contain code to authorizes API calls. Apart from that, it returns search results as expected :)

I was not sure which could be the best authorization mechanism, so I didn't implement it. Also, it is not possible to use Meteor.user() on server side iron router for authorization. See yourself: http://stackoverflow.com/questions/20219572/meteor-user-on-iron-router-server-side

PS: I have rebased ESIntegration branch on search_api.

@D-Ne0 D-Ne0 changed the title from Search api to Search API May 30, 2014

@rtnpro

This comment has been minimized.

Show comment
Hide comment
@rtnpro

rtnpro Jun 1, 2014

Member

@D-Ne0 Please rebase your codebase on top of latest master.

Member

rtnpro commented Jun 1, 2014

@D-Ne0 Please rebase your codebase on top of latest master.

@D-Ne0

This comment has been minimized.

Show comment
Hide comment
@D-Ne0

D-Ne0 Jun 4, 2014

Contributor

Integrated front-end code with API.

Contributor

D-Ne0 commented Jun 4, 2014

Integrated front-end code with API.

@D-Ne0 D-Ne0 changed the title from Search API to Search API, UI and their integration Jun 4, 2014

@rtnpro

This comment has been minimized.

Show comment
Hide comment
@rtnpro

rtnpro Jul 27, 2014

Member

Closing this as the changes have been pulled in bookmarking branch.

Member

rtnpro commented Jul 27, 2014

Closing this as the changes have been pulled in bookmarking branch.

@rtnpro rtnpro closed this Jul 27, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment