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

3 1 5 backport #37

Open
wants to merge 36 commits into
base: 3-1-stable
Choose a base branch
from
Open

3 1 5 backport #37

wants to merge 36 commits into from

Conversation

tanmay3011
Copy link
Member

No description provided.

tanmay3011 and others added 20 commits June 27, 2016 10:55
Update gem name from spree_reportify to spree_admin_insights
…issue

Get db config from ActiveRecord::Base.
changes params for Sequel connection(for sqlite db)
1. Standardize on report_category instead of using type and category alternatively.
2. Add initializer to configure list of reports to be used.
3. Reduce responsibility of report generation service to just generate reports
1. There was an issue with order dependency while populating the report
data back into timed observations, reverting to using a search and fill
strategy instead of filling them sequentially.
Shipment may lie in an entirely different time period then that of
when the order was completed. The date used for considering an order is it’s
completed_at date, Using the same date for calculating shipment ensures that they 
lie within the expected reporting period.
@tanmay3011 tanmay3011 mentioned this pull request Jul 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants