Skip to content
This repository has been archived by the owner on Oct 18, 2018. It is now read-only.

Email users who haven't logged in for a while (Issue #25) #71

Closed
wants to merge 11 commits into from
Closed

Email users who haven't logged in for a while (Issue #25) #71

wants to merge 11 commits into from

Conversation

steven-ferguson
Copy link
Contributor

#25

@@ -180,6 +204,16 @@ def organizer?(event)
organizer.user_uuid == user_uuid
end
end

def update_attribute(attribute_hash)

Choose a reason for hiding this comment

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

This should probably be called update_attributes.

@ErinCall
Copy link

@steven-ferguson this is great and I'm gonna steal it! I'm going to close this pull request and address the code review notes on my own fork :~)

@ErinCall ErinCall closed this Jan 23, 2014
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