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.

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

Jump to bottom

Improvements #9

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

Improvements #9

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
8274bc3
Change birthdays database to YAML / JSON
shinenelson May 28, 2017
17a3ea0
Configurize db_path for the birthdays database
shinenelson May 28, 2017
726d61a
Rename configurations.json to configuration.json.example
shinenelson May 28, 2017
c45e1e1
Optimize bot to work with {hash}ed birthdays
shinenelson May 28, 2017
9f7bc88
Configurize mention'ing team members
shinenelson Jun 4, 2017
dfbcc78
Update README.md with more detailed instructions
shinenelson Jun 4, 2017
d409122
Rename db_path to birthdays_path (Resolves https://github.com/jeKnowl…
shinenelson Jun 4, 2017
077a104
Log Time.now
shinenelson Jun 4, 2017
2ee35b3
change date keys to string
shinenelson Aug 1, 2017
9b83de9
handle empty / nil YAML.load_file
shinenelson Aug 1, 2017
ed5c9cb
remove spaces (Resolves https://github.com/jeKnowledge/slack-birthday…
shinenelson Aug 3, 2017
e901b71
move birthday list filter to birthday_bot from birthday_reader
shinenelson Aug 7, 2017
f4dbfe8
add today = Time.now (Resolves https://github.com/jeKnowledge/slack-b…
shinenelson Aug 7, 2017
e3a0763
def build_user_list
shinenelson Aug 7, 2017
56c8b07
[refactor] remove httparty dependency
shinenelson Nov 21, 2019
e64afd9
[docs] replace db_path with birthdays_path
shinenelson Nov 21, 2019
0596516
[readme] fix URL in contributors' list
shinenelson Nov 21, 2019
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
1 change: 1 addition & 0 deletions configurations.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
{
"db_path": "birthdays.yaml",
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think it makes more sense to change db_path to birthdays_path

"slack_url": "https://hooks.slack.com/...",
"channel_name": "general",
"greeting_message": "Happy birthday !",
Expand Down
3 changes: 1 addition & 2 deletions lib/birthday_bot.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,15 +3,14 @@
require 'birthday_reader'

class BirthdayBot
@@path = "birthdays.txt"

def initialize()
@config = ConfigReader.new
@config.load('configurations.json')
end

def start!
birthdays = BirthdayReader.get_birthdays(@@path)
birthdays = BirthdayReader.get_birthdays(@config.db_path)
today = Time.now

puts "Checking who was born today (#{today.to_s})"
Expand Down
4 changes: 3 additions & 1 deletion lib/config_reader.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,8 @@
require 'json'

class ConfigReader
attr_accessor :slack_url,
attr_accessor :db_path,
:slack_url,
:channel_name,
:greeting_message,
:bot_name,
Expand All @@ -11,6 +12,7 @@ def load(filename)
if File.exist?(filename)
file = File.read(filename)
config = JSON.parse(file)
@db_path = config['db_path']
@slack_url = config['slack_url']
@channel_name = config['channel_name']
@greeting_message = config['greeting_message']
Expand Down