Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Adds Setting model and interface to manage settings #142

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Contributor

danmcclain commented Jan 3, 2012

The Setting model provides a dictionary for values. It can be used to
store content of the site that change from time to time (like Venue,
Sign up link, etc).

The Venue name, map link, address and sign up link have been updated to
use the Setting model. This commit includes a migration with the values
for the January meeting.

The Setting model can be used the following way

  • Setting['some_key'] = 'some value' # Sets the value of some_key,
    creating the setting if necessary
  • Setting['some_key'] # => 'some value'

Proper testing of Setting and the admin interface have been added

Settings admin interface available at /admin/settings

Adds AdminController which serves as base class of
Admin::PresentationController and Admin::SettingController.
AdminController contains the logic needed to force http basic
authentication.

Dan Seaver Adds Setting model and interface to manage settings
The Setting model provides a dictionary for values. It can be used to
store content of the site that change from time to time (like Venue,
Sign up link, etc).

The Venue name, map link, address and sign up link have been updated to
use the Setting model. This commit includes a migration with the values
for the January meeting.

The Setting model can be used the following way

* Setting['some_key'] = 'some value' # Sets the value of some_key,
  creating the setting if necessary
* Setting['some_key'] # => 'some value'

Proper testing of Setting and the admin interface have been added

Settings admin interface available at /admin/settings

Adds AdminController which serves as base class of
Admin::PresentationController and Admin::SettingController.
AdminController contains the logic needed to force http basic
authentication.
dcdff00
Contributor

bcardarella commented Jan 5, 2012

While the code looks good I question the necessity of implementing a key value store in AcitveRecord. Couldn't we just serialize a column?

Contributor

danmcclain commented Feb 5, 2012

I'm still planning on updating this to serialize a column, been a busy month!

@danmcclain danmcclain closed this Sep 5, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment