webbrowser-app

Merge lp://qastaging/~fboucault/webbrowser-app/add_peekier_search into lp://qastaging/webbrowser-app/staging

  1. add_peekier_search
  2. Merge into staging
Proposed by Florian Boucault
Status: Merged
Approved by: Olivier Tilloy
Approved revision: 1607
Merged at revision: 1607
Proposed branch: lp://qastaging/~fboucault/webbrowser-app/add_peekier_search
Merge into: lp://qastaging/webbrowser-app/staging
Diff against target: 14 lines (+10/-0)
1 file modified
src/app/webbrowser/searchengines/peekier.xml (+10/-0)
To merge this branch: bzr merge lp://qastaging/~fboucault/webbrowser-app/add_peekier_search
Reviewer Review Type Date Requested Status
Olivier Tilloy Approve
Review via email: mp+317206@code.qastaging.launchpad.net

Commit message

Add Peekier to available search engines.

Description of the change

Add Peekier to available search engines.

To post a comment you must log in.
Revision history for this message
Olivier Tilloy (osomon) wrote :

LGTM.

It looks like we could get search suggestions as we type from https://search.peekier.com/suggestions, but that appears to work only with POST requests, and the opensearch specification doesn't seem to allow that (nor does our implementation).

Plain searches will be good enough for now.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches