-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
edit: new indexes config #7639
edit: new indexes config #7639
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good @bholmesdev ! Just a few quick things below. You know the drill: if it's not released yet, add the merge on release
label. If it is, go ahead and merge when you're happy!
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
@bholmesdev Just a quick check that this looks like it can be merged now? Normally we let maintainers keep track of their own feature docs that aren't part of a coordinated minor effort. Happy to merge this for you if needed, but this is also the kind of thing you can come and merge when your patch goes out! |
Ah! This should have been merged a while ago. Yes plesae |
Description (required)
Update
indexes
configuration from an object to an array. Also note that index names are generated with aname
property to override.Related issues & labels (optional)