Skip to content

Implementation to aggregate statistical information #12 #128

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.

Sign up for GitHub

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

Merged
merged 37 commits into from
Aug 21, 2017
Merged

Implementation to aggregate statistical information #12 #128

Changes from 1 commit
Commits
Show all changes
37 commits
Select commit Hold shift + click to select a range
e3fd895
Gem `faraday`
nalabjp Jul 29, 2017
db0afac
Draft implementaion of API client
nalabjp Jul 30, 2017
04f645c
Create a table named `event_histories`
nalabjp Jul 30, 2017
6ff7f21
Create a model named `EventHistory`
nalabjp Jul 30, 2017
453d28f
Create a table named `dojo_event_services`
nalabjp Jul 30, 2017
3669f2a
Create a model named `DojoEventService`
nalabjp Jul 30, 2017
b3d39bc
Add associations
nalabjp Jul 30, 2017
f3580de
Add timezone as `Asia/Tokyo`
nalabjp Jul 30, 2017
3bc50c1
Draft implementation of importing data via API to `event_histories` t…
nalabjp Jul 30, 2017
dfdc16c
Fix
nalabjp Aug 6, 2017
c93dfb8
Support multiple requests
nalabjp Aug 6, 2017
9eb61aa
Support a period for search conditions
nalabjp Aug 6, 2017
a4cb3ec
Support idempotency in re-running with same conditions
nalabjp Aug 6, 2017
d75d2fe
Rename a class name
nalabjp Aug 6, 2017
8ba4506
Use instance variables instead of constants
nalabjp Aug 6, 2017
b1a5ad1
Fix typo
nalabjp Aug 6, 2017
7197558
Create a task for the monthly aggregation
nalabjp Aug 6, 2017
983d828
Want to see some search results
nalabjp Aug 13, 2017
cf033f8
Rake task for searching
nalabjp Aug 13, 2017
70ef722
Fix typo
nalabjp Aug 13, 2017
5168879
Migrate database on Travis CI
nalabjp Aug 13, 2017
ba6fa8f
Run RSpec on Travis CI
nalabjp Aug 13, 2017
59fe612
Ruby 2.4.0 on Travis CI
nalabjp Aug 13, 2017
ec719e1
Cache bundler on Travis CI
nalabjp Aug 13, 2017
0024192
Shared context for API stubs
nalabjp Aug 13, 2017
b7e8213
Remove nested Hash key
nalabjp Aug 13, 2017
58bfebd
Add testing for `Statistics::Client`
nalabjp Aug 13, 2017
ab9736d
Add testing for `Statistics::Aggregation`
nalabjp Aug 13, 2017
740ca73
Remove records of the target period before executing the aggregation
nalabjp Aug 13, 2017
86b1f6c
Add description for `statistics:search`
nalabjp Aug 13, 2017
563bd68
Remove needless configuration
nalabjp Aug 14, 2017
f44f78e
Remove needless configuration
nalabjp Aug 14, 2017
0d72302
Fix association name
nalabjp Aug 14, 2017
754cf52
Remove needless argument
nalabjp Aug 20, 2017
0366800
Move to `.travis.yml`
nalabjp Aug 20, 2017
9adb714
Add `NOT NULL` restriction
nalabjp Aug 20, 2017
d392c68
Change the column type of `dojo_event_services.name` to `integer` fro…
nalabjp Aug 20, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions config/application.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,5 +11,9 @@ class Application < Rails::Application
# Settings in config/environments/* take precedence over those specified here.
# Application configuration should go into files in config/initializers
# -- all .rb files in that directory are automatically loaded.

# Timezone
config.time_zone = 'Asia/Tokyo'
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

ENV['TZ'] = 'Asia/Tokyo'
Copy link
Contributor

Choose a reason for hiding this comment

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

Heroku、travis-ciのほうで設定した方がいいかも?

Copy link
Member Author

Choose a reason for hiding this comment

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

Herokuのenvはどなたかにお願いするとして、そうしましょうか。

Copy link
Member

Choose a reason for hiding this comment

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

Heroku の env 確認しましたが、既に入ってました ;) ✅

$ heroku config
...
TZ:                       Asia/Tokyo

Copy link
Member Author

Choose a reason for hiding this comment

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

ご確認ありがとうございます！😁

Copy link
Member Author

Choose a reason for hiding this comment

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

ご確認ありがとうございます！😁

Copy link
Member Author

Choose a reason for hiding this comment

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

fixed 0366800

end
end