Skip to content
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

Rebranding name change #3370

Merged
merged 5 commits into from
Mar 17, 2015
Merged

Rebranding name change #3370

merged 5 commits into from
Mar 17, 2015

Conversation

panda01
Copy link
Contributor

@panda01 panda01 commented Mar 17, 2015

Changes necessary from the Keynote rebranding

panda01 added 2 commits March 16, 2015 18:22
… string elasticsearch to take into consideration the rebranding during elasitcon and keynote
@jordansissel
Copy link
Contributor

Nice work! This reminds me that we have to do this for Logstash, too.

@panda01 panda01 assigned simianhacker and unassigned spalger Mar 17, 2015
@@ -7,7 +7,7 @@
<div>

<p>
By default, Elasticsearch scripts use <a target="_window" href="http://www.elasticsearch.org/guide/en/elasticsearch/reference/current/modules-scripting.html#_lucene_expressions_scripts">Lucene Expressions <i class="fa-link fa"></i></a>, which is a lot like JavaScript, but limited to basic arithmetic, bitwise and comparison operations. We'll let you do some reading on <a target="_window" href="http://www.elasticsearch.org/guide/en/elasticsearch/reference/current/modules-scripting.html#_lucene_expressions_scripts">Lucene Expressions<i class="fa-link fa"></i></a> To access values in the document use the following format:
By default, Elasticsearch scripts use <a target="_window" href="http://www.elastic.co/guide/en/elasticsearch/reference/current/modules-scripting.html#_lucene_expressions_scripts">Lucene Expressions <i class="fa-link fa"></i></a>, which is a lot like JavaScript, but limited to basic arithmetic, bitwise and comparison operations. We'll let you do some reading on <a target="_window" href="http://www.elastic.org/guide/en/elasticsearch/reference/current/modules-scripting.html#_lucene_expressions_scripts">Lucene Expressions<i class="fa-link fa"></i></a> To access values in the document use the following format:
Copy link
Member

Choose a reason for hiding this comment

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

http://www.elastic.org/guide/en/elasticsearch/reference/current/modules-scripting.html#_lucene_expressions_scripts

should be

http://www.elastic.co/guide/en/elasticsearch/reference/current/modules-scripting.html#_lucene_expressions_scripts

@simianhacker simianhacker assigned panda01 and unassigned simianhacker Mar 17, 2015
@panda01
Copy link
Contributor Author

panda01 commented Mar 17, 2015

Addressed @simianhacker's concerns.

@panda01 panda01 assigned simianhacker and unassigned panda01 Mar 17, 2015
@simianhacker
Copy link
Member

LGTM

panda01 added a commit that referenced this pull request Mar 17, 2015
@panda01 panda01 merged commit 9c98861 into elastic:master Mar 17, 2015
spalger pushed a commit to spalger/kibana that referenced this pull request Mar 23, 2015
panda01 added a commit that referenced this pull request Mar 23, 2015
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.

4 participants