| Commit message (Collapse) | Author | Age | Lines |
|
|
|
|
|
|
|
|
|
|
|
| |
Factories are loaded automatically from:
- test/factories.rb
- spec/factories.rb
- test/factories/*.rb
- spec/factories/*.rb
Keeping factories per-model helps navigation, especially when they get
more complex with traits and inheritance.
|
|\ |
|
| |
| |
| |
| | |
Get the canonical human-readable status of a comment
|
| | |
|
| | |
|
|/
|
|
| |
The UI provides a list of comment_ids and a newstate="visible" or "hide"
|
|\ |
|
| |
| |
| |
| | |
Closes #1313 and #365.
|
| |
| |
| |
| | |
The InfoRequest is always the one associated with the event.
|
| |
| |
| |
| | |
Text will be escaped by default in Rails 3
|
|\ \
| | |
| | |
| | | |
rails-3-develop
|
|/ / |
|
| | |
|
| | |
|
|\ \
| | |
| | |
| | | |
'opennewzealand_github/feature/handle-undefined-conversion' into rails-3-develop
|
| | | |
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | | |
From: http://ruby-doc.org/core-2.0/String.html#method-i-encode
Ducktypes for having encode rather than relying on RUBY_VERSION
|
|\ \ \
| | | |
| | | |
| | | | |
ssh://git.mysociety.org/data/git/public/alaveteli into rails-3-develop
|
| |\ \ \ |
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Extract checking whether a user is banned from making Comments on an
InfoRequest to a filter in CommentController.
Removes responsibility from the #new method.
Adds a missing spec.
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Extract checking whether comments are allowed on an InfoRequest to a
filter in CommentController.
Removes responsibility from the #new method.
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Use a before_filter to make @track_thing available to all filters
called on the same action and remove responsibility from the #new method
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Use a before_filter to make @info_request available to all filters
called on the same action
|
| |/ / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Fixes https://github.com/mysociety/alaveteli/issues/662
If /annotate/request/:url_title is accessed when comments are disabled
an exception is incorrectly thrown.
Conditionals should be used for control flow, so now the action
redirects to the info_request path and displays a notice.
|
|\ \ \ \
| |/ / /
|/| | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
We only really want to redirect people to live sites, so pre-launch
sites don't belong in WorldFOIWebsites. Handle the case where the
current locale isn't there. Closes #1345.
|
|\ \ \ \ |
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Calling simple_date threw an exception as it uses a helper internally.
Only LinkToHelper is included in to the controller, so the underlying
helper was not available.
|
|/ / / / |
|
|\ \ \ \ |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
For a better workflow
http://mysociety.github.io/installation-standards.html#silver
# Host
# Note that --recursive must be specified to include commonlib
# submodule
$ git clone --recursive git@github.com:mysociety/alaveteli.git
$ cd alaveteli && vagrant up
$ vagrant ssh
# Guest
$ cd /home/vagrant/alaveteli && bundle exec rake spec
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Background IRC Chat:
2014-03-07T11:28:33+0000] <gareth> Why is the branch being replaced
here?
https://github.com/mysociety/alaveteli/blob/rails-3-develop/config/Vagra
ntfile#L18
[2014-03-07T11:29:55+0000] <gareth> It can't checkout install-script
branch as it doesn't exist
[2014-03-07T11:32:10+0000] <gareth> not sure how/why it could do this
when it fetched a new copy of the repo rather than using the host
checkout
[2014-03-07T11:35:42+0000] <louise> gareth that may be my fault - I
deleted a bunch of what I thought were old branches earlier this week -
perhaps install-script was one of them - let me look
[2014-03-07T11:36:58+0000] <louise> nope, doesn't look like it was -
mark, should the install-script branch exist?
[2014-03-07T11:38:18+0000] <mark> what the install script does is to
use the first of these that can be checked out: origin/install-script
0.15 origin/master
[2014-03-07T11:38:46+0000] <mark> but i must have created that
Vagrantfile while the install-script branch existed and worked, but the
master branch wouldn't
[2014-03-07T11:38:56+0000] <mark> i think now it'd be safe to take that
line out of the vagrant file
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Share the host git checkout in to the VM so that a developer can use
their preferred environment while running the application in a
production-like environment.
Use the --dev option of install-site.sh so that the existing git checkout is
used.
The gems are installed outside of the app directory in development
because VirtualBox's directory shares can be slow when the directory
contains an excessive amount of files. Installing gems in to this
directory results in ~15K additional files, which is beyond what
VirtualBox can reasonably be expected to handle
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
- Sets vendor/bundle as default
- Uses $HOME/bundle when generating general.yml
Based on d9f162c8ab08cb881cd68a3eee900c52c63cb1ae
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
|/ / / /
| | | |
| | | |
| | | |
| | | |
| | | | |
Required for Vagrant >=1.1
Should be backwards compatible with older versions
http://docs.vagrantup.com/v2/installation/backwards-compatibility.html
|
|\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Conflicts:
config/general.yml-example
locale/cy/app.po
|
| | | | | |
|
| | | | | |
|
| |\ \ \ \ |
|
| | | | | | |
|
| |\ \ \ \ \
| | | |_|/ /
| | |/| | |
| | | | | |
| | | | | | |
Conflicts:
locale/cy/app.po
|