Skip to content
This repository was archived by the owner on Mar 7, 2018. It is now read-only.

Conversation

@c-w
Copy link
Contributor

@c-w c-w commented Aug 22, 2017

In the future we want to remove this hard-coded value, see #130 for tracking issue.

In the future we want to remove this hard-coded value, see
#130
for tracking issue.
@c-w c-w requested a review from erikschlegel August 22, 2017 16:54
Copy link
Contributor

@erikschlegel erikschlegel left a comment

Choose a reason for hiding this comment

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

LGTM

@c-w c-w merged commit 32e8c17 into master Aug 22, 2017
@c-w c-w deleted the fix-create-site branch August 22, 2017 18:27
@c-w c-w removed the in progress label Aug 22, 2017
rachelnicole pushed a commit that referenced this pull request Jan 24, 2018
* Fixed admin setting integration issues

* A variety of bug fixes to support pulling fortis events from the cassandra table into the news feed.

* A variety of fixes to support showing fortis events in the news feed.

* Fixing linting errors

* Reverting executeQuery change

* Adding avgsentiment calc

* Reorganizing aggregation logic

* Reorganizing aggregation logic

* Refactored services to integrate reoptimized cassandra tables

* Fixing lint errors

* Fixing lint errors

* Adding tile id list service for heatmap

* Rebased

* Rebasing

* Rebasing
rachelnicole pushed a commit that referenced this pull request Jan 24, 2018
Changing eventid to follow <pipelinekey>.<eventsourceid> pattern.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants