-
Notifications
You must be signed in to change notification settings - Fork 536
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
Add AutocompleteContext to the Autocomplete component's exports #2115
Add AutocompleteContext to the Autocomplete component's exports #2115
Conversation
🦋 Changeset detectedLatest commit: 5b12b0a The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
@mperrotti it looks like you might need to kick off some CI? |
@colebemis - the new CI issue with package-lock.json is happening. How did you get it to work last time? Here's what I've tried:
cc @siddharthkp since you're not as busy with the design summit. |
@willglas - Could you open a PR as a branch directly off of this repo? Org tokens aren't propagating or being respected, so deployments aren't working for PRs from a forked repo. |
@mperrotti I don't have permissions to push branches directly to this repo. Do you know how I can get it? |
Hey @willglas - I’m looking into this now. Worst-case scenario, I’ll cherry-pick your commits onto a new branch and open a PR |
@willglas - you should have received an invite |
@mperrotti thanks! here's the new PR: #2153 |
Added
AutocompleteContext
to theAutocomplete
component's exports. This allows consumers to customize theAutocomplete
component's behavior to a much larger degree.Screenshots
No visual changes.
Merge checklist
I wasn't sure if updating documentation was necessary, let me know if that's required for this change to go in.