Skip to content

Conversation

@Aschen
Copy link
Contributor

@Aschen Aschen commented Mar 27, 2019

What does this PR do?

Replace references to data collection and data index by index and collection

@Aschen Aschen self-assigned this Mar 27, 2019
@kuzzle
Copy link
Contributor

kuzzle commented Mar 27, 2019

Snippets tests reports

SDK Version Tests overview
v6 ✔️
v5 ✔️
v1 ✔️
v1 ✔️
v1 ✔️
v1 ✔️


// Freshly installed Kuzzle servers are empty: we first need to create
// a data index. The one used in this example is named "nyc-open-data"
// a index. The one used in this example is named "nyc-open-data"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are other occurences of this error, and they should all be fixed.

Suggested change
// a index. The one used in this example is named "nyc-open-data"
// an index. The one used in this example is named "nyc-open-data"

index.js Outdated

if (options.algolia.privateKey) {
log('Algolia indexing enabled');
log('Algolian indexing enabled');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I... think that you should check your regexes before applying them. 😑

@scottinet scottinet merged commit 96c6e6b into master Mar 28, 2019
@scottinet scottinet deleted the remove-data-collection branch March 28, 2019 09:58
@Aschen Aschen mentioned this pull request May 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants