Commit message (Collapse) | Author | Age | Lines | |
---|---|---|---|---|
* | Handle case where info request doesn't have a user_name | Louise Crow | 2013-01-07 | -0/+8 |
| | ||||
* | Handle the case of a comment on an external request. | Louise Crow | 2012-12-21 | -1/+1 |
| | ||||
* | Don't require rspec in production. | Louise Crow | 2012-12-18 | -2/+0 |
| | ||||
* | If we're not going to run the commonlib tests (and I think that's correct - ↵ | Louise Crow | 2012-12-18 | -3/+0 |
| | | | | they belong in commonlib and should only be run when commonlib is changed), then don't load them. Allows us not to include the rspec gem in production. | |||
* | Include Rake::DSL module manually to avoid deprecation warnings as per | Louise Crow | 2012-12-18 | -0/+3 |
| | | | | https://github.com/dchelimsky/rspec-rails/commit/be52a9841dde0292c7bc7ecfe39e7214b4db1cab | |||
* | Latest translation files re-pulled from Transifex to get rid of fuzzy strings. | Louise Crow | 2012-12-18 | -963/+467 |
| | ||||
* | Latest translations from Transifex. | Louise Crow | 2012-12-18 | -11/+3926 |
| | ||||
* | Limit pagination on similar pages in line with new upper limit on page offset. | Louise Crow | 2012-12-17 | -1/+3 |
| | ||||
* | Only serve up 'similar' pages up to the offset we use for list. | Louise Crow | 2012-12-17 | -0/+13 |
| | ||||
* | Don't test travis fix branch anymore. | Louise Crow | 2012-12-17 | -1/+0 |
| | ||||
* | Don't install the develop or development group of gems in travis. | Louise Crow | 2012-12-17 | -0/+4 |
| | ||||
* | Add a spec to test that mailin does not produce any output. This is ↵ | Louise Crow | 2012-12-17 | -0/+21 |
| | | | | important as under some MTA configurations, any output will result in a bounce message being sent to the original mail sender. | |||
* | Adding mocking of incoming message id. | Louise Crow | 2012-12-17 | -0/+1 |
| | ||||
* | Don't install packages from the package list that we've already installed ↵ | Louise Crow | 2012-12-17 | -1/+1 |
| | | | | another way. | |||
* | Don't install rubygems - just need to update. However, seems like you can't ↵ | Louise Crow | 2012-12-17 | -1/+1 |
| | | | | downgrade from the current rubygems under ruby 1.9.3 (1.8.24) to 1.6.2 in one go :( - add an intermediate version. | |||
* | Actually the 1st Nov, not 15th. | Louise Crow | 2012-12-17 | -1/+1 |
| | ||||
* | Send the annual bank holiday reminder half-way through November, not on the ↵ | Louise Crow | 2012-12-17 | -1/+1 |
| | | | | 1st of January, so there's actually time to act on it before the usual Jan 1 Bank Holiday being missing starts resulting in incorrect due dates. | |||
* | Rewrite specs that were in spec/controller/application_controller as ↵ | Louise Crow | 2012-12-17 | -56/+71 |
| | | | | full-stack controller specs in the relevant controllers. It turns out that having spec blocks that reference the ApplicationController class directly i.e. "describe ApplicationController" can have unpredictable effects. actionpack's action_controller/test_case.rb rewrites rescue_action_without_handler on whatever it is included in, and if this is done on a controller class, and then directly on action controller, it can result in an infinite loop of recursive calls. This turns out to be the problem that was causing some tests in error_spec.rb to fail in Travis under Ruby 1.9. | |||
* | Merge branch 'hotfix/authenticate-and-expire-download-zips' into develop | Louise Crow | 2012-12-13 | -13/+67 |
|\ | ||||
| * | Remove any download zip files when a request's cached files are expired. | Louise Crow | 2012-12-13 | -0/+8 |
| | | ||||
| * | Put download zips in a predictable location - sharded folders based on ↵ | Louise Crow | 2012-12-13 | -1/+9 |
| | | | | | | | | request ID, rather than distributing them across the download directories by the generated SHA. Preserve the uniqueness of the subdirectory. | |||
| * | Use helper method for download path, and use Rails.root instead of file ↵ | Louise Crow | 2012-12-13 | -1/+5 |
| | | | | | | | | location. | |||
| * | Check that a request is publicly visible before generating a download link. | Louise Crow | 2012-12-13 | -10/+26 |
| | | ||||
| * | Only offer download links on publicly visible requests | Louise Crow | 2012-12-13 | -1/+19 |
| | | ||||
| * | Merge branch 'hotfix/no-html-conversion-for-hidden'0.6.8.1 | Louise Crow | 2012-12-13 | -8/+29 |
| |\ | ||||
* | | | Updated lock file. | Louise Crow | 2012-12-13 | -0/+1 |
| | | | ||||
* | | | Restore rspec - causing errors on production deploy with cap. | Louise Crow | 2012-12-13 | -0/+1 |
| | | | ||||
* | | | Merge branch 'hotfix/no-html-conversion-for-hidden' into develop | Louise Crow | 2012-12-13 | -3/+18 |
|\ \ \ | | |/ | |/| | ||||
| * | | Don't offer or allow viewing of an HTML version of a response attachment if ↵ | Louise Crow | 2012-12-13 | -8/+29 |
| |/ | | | | | | | the request is hidden, or requester_only. Google docs viewer won't be able to access it, and our own conversion process currently can produce image files that will then be publicly viewable directly from the webserver (see config/httpd.conf). If necessary we can revisit this code to enable admins and requesters to view the HTML version created by our own conversion without adding these files to a path that is served directly by the web server. | |||
| * | Emphasise the importance of adding a newrelic.yml file, fix formatting. | Louise Crow | 2012-11-12 | -1/+2 |
| | | ||||
| * | Merge remote-tracking branch 'origin/release/0.6.8'0.6.8 | Louise Crow | 2012-11-12 | -8979/+9638 |
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: locale/cs/app.po locale/sl/app.po locale/sq/app.po locale/tr/app.po locale/uk/app.po | |||
| * \ | Merge branch 'hotfixes/translations0.6.7' | Louise Crow | 2012-11-06 | -316/+8006 |
| |\ \ | ||||
| | * | | Latest translations from Transifex. | Louise Crow | 2012-11-06 | -316/+8006 |
| |/ / | ||||
| * | | Require rather than load to avoid already defined constant error when ↵0.6.7 | Louise Crow | 2012-10-17 | -1/+1 |
| | | | | | | | | | | | | loading rails environment. | |||
| * | | Merge branch 'release/0.6.7' | Louise Crow | 2012-10-17 | -809/+2629 |
| |\ \ | | | | | | | | | | | | | | | | | | | | | Conflicts: locale/cs/app.po locale/ro_RO/app.po | |||
| * | | | Revert "Revert "Merge remote-tracking branch 'henare_github/patch-1'"" | Louise Crow | 2012-10-17 | -86/+156 |
| | | | | | | | | | | | | | | | | This reverts commit 211fe84dc40d97df8aa8724906d9170ed4f78477. | |||
| * | | | Latest translations from Transifex. | Louise Crow | 2012-10-11 | -129/+3931 |
| | | | | ||||
| * | | | Revert "Merge remote-tracking branch 'henare_github/patch-1'" | Louise Crow | 2012-10-09 | -156/+86 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | Mistakenly merged into master. Please note that we'll want to merge this work into master on the next release, at which point we'll want to revert this reversion in order that the changes are properly re-applied. This reverts commit 54281fd50c3271835a54ab4bc08d40da09d643ee, reversing changes made to 793ca358c37458e6cc4385d2366621aaee93a25e. | |||
| * | | | Merge remote-tracking branch 'henare_github/patch-1' | Louise Crow | 2012-09-24 | -86/+156 |
| |\ \ \ | ||||
* | | | | | Reformat for line length, add the expected response code. | Louise Crow | 2012-12-13 | -2/+9 |
| | | | | | ||||
* | | | | | Don't offer or allow viewing of an HTML version of a request if it is ↵ | Louise Crow | 2012-12-13 | -8/+29 |
| | | | | | | | | | | | | | | | | | | | | hidden, or requester_only. Google docs viewer won't be able to access it, and our own conversion process currently produces image files that will then be publicly viewable. If necessary we can revisit this code to enable admins and requesters to view the HTML version created by our own conversion without adding these files to a path that is served directly by the web server. | |||
* | | | | | Merge remote-tracking branch 'henare_github/rspec' into develop | Louise Crow | 2012-12-12 | -3/+0 |
|\ \ \ \ \ | ||||
| * | | | | | Don't install rspec in production | Henare Degan | 2012-12-11 | -3/+0 |
| | | | | | | | | | | | | | | | | | | | | | | | | Also don't need to explicitly specify the rspec gem as rspec-rails depends on it | |||
* | | | | | | Merge branch 'feature/set-zone-in-tests' into develop | Louise Crow | 2012-12-12 | -50/+61 |
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | ||||
| * | | | | | Wrap specs on the extraction of RFC-822 headers in code that sets the ENV ↵ | Louise Crow | 2012-12-11 | -50/+61 |
|/ / / / / | | | | | | | | | | | | | | | | timezone. TMail renders headers using localtime, which is not ideal, but we're migrating away from it anyway, so I'm not sure it's worth delving into the internals of TMail to fix it. | |||
* | | | | | Merge remote-tracking branch 'henare_github/relax-gc-for-tests' into develop | Louise Crow | 2012-12-10 | -0/+14 |
|\ \ \ \ \ | ||||
| * | | | | | Only trigger garbage collection every 4 seconds - speeds up the whole test ↵ | Henare Degan | 2012-12-08 | -0/+14 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | suite by ~35% on my machine Via https://37signals.com/svn/posts/2742-the-road-to-faster-tests | |||
* | | | | | | Merge remote-tracking branch 'henare_github/move-gem' into develop | Louise Crow | 2012-12-10 | -1/+1 |
|\ \ \ \ \ \ | ||||
| * | | | | | | Don't install test-unit in production | Henare Degan | 2012-12-09 | -1/+1 |
| |/ / / / / | ||||
* | | | | | | Merge remote-tracking branch 'henare_github/remove-vendored-gem' into develop | Louise Crow | 2012-12-10 | -6253/+0 |
|\ \ \ \ \ \ |