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.

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

Switch upstream to corejavascript fork, merge #8

Merged
merged 97 commits into from Nov 29, 2016

Conversation

Projects
None yet
@colinrotherham

colinrotherham commented Nov 29, 2016

This pull request adds https://github.com/corejavascript/typeahead.js as the upstream fork.

With the additions:

  1. ARIA changes have been preserved
  2. Packages updated
  3. All tests fixed and passing

This gives us a new stable base.

After this pull, consider opening another pull request to corejavascript/typeahead.js so these ARIA improvements can be patched upstream as well.

wakingrufus and others added some commits Sep 2, 2014

Update highlight.js
made diacriticInsensitive an option that can be set instead of default behavior
Merge pull request twitter#1199 from jharding/1163-val-undefined
Coerce val to string when set through API
Merge pull request twitter#1200 from jharding/1185-bad-async-logic
Fix bad async suggestions rendering logic
Adding Option: matchAnyQueryToken (default false)
- By default a search result must match ALL query-tokens. Instead, this option returns results that match ANY
ApfelUser ApfelUser
Merge pull request #4 from corejavascript/integration-0.11.2
Merge previous work on 0.11.2 into master
ApfelUser ApfelUser
Merge pull request #2 from chip-miller/integration-0.11.2
Option: matchAnyQueryToken (default false)
Update rendered-count _after_ async results have displayed
This ensures that all async results are rendered.
ApfelUser ApfelUser
Merge pull request #8 from coopy/fix-async-render-corejs
Update rendered-count _after_ async results have displayed
Ben Creasy
bloodhound.get->bloodhound.search
Lots of issues about this in the twitter repo.
ApfelUser ApfelUser
Merge pull request twitter#14 from jcrben/patch-1
bloodhound.get->bloodhound.search
node-static webserver installed locally is good enough.
No need to install that dependency globally
Travis must have bower installed before running "npm install"
Since the `npm install` job runs `bower install` during the `postinstall` phase.

Also note that `bower install` is no longer necessary to specify in
`.travis.yml`
when it's inside the package.json postinstall step
ApfelUser ApfelUser
Merge pull request twitter#15 from jesperronn/npm_postinstall
Npm postinstall step to npm to cater for easy setup

jlbooker and others added some commits Oct 24, 2016

Merge pull request twitter#74 from mattiasrunge/master
Removed requirement to have bower when installing with NPM
Implement edge ngram tokenizer for Bloodhound (twitter#93)
Add tests for ngram tokenizer
Add tokenizer examples to Bloodhound readme
Build dist
NPM & Bower version patch to 0.11.2
Merge pull request twitter#62 from sbsurf/master
Allowing for custom setting of maxPendingRequests.
Merge pull request twitter#51 from wakingrufus/master
diacritic-insensitive highlighting
Merge pull request twitter#100 from jorrit/patch-1
Remove non-essential files from npm package
Update status parameters
Fixes result count in ARIA status message
Update packages
Also fix broken promises tests
Release 1.0.3
Adds notes regarding original fork and move to new upstream: https://github.com/corejavascript/typeahead.js
@colinrotherham

This comment has been minimized.

Show comment
Hide comment
@colinrotherham

colinrotherham Nov 29, 2016

As mentioned in the description above, another pull request has been opened with the upstream repo: corejavascript#104

colinrotherham commented Nov 29, 2016

As mentioned in the description above, another pull request has been opened with the upstream repo: corejavascript#104

@easternbloc easternbloc merged commit a151a20 into UKHomeOffice:master Nov 29, 2016

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