codeforamerica / ohana-web-search
90%
master: 98%

LAST BUILD BRANCH: mb-circle-ci
DEFAULT BRANCH: master
Repo Added
Files 26
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH 336-remove-hardcoded-smc-code
branch: 336-remove-hardcoded-smc-code
CHANGE BRANCH
x

1367

push

travis-ci

monfresh 
Improve code, fix bugs, and updates specs in PR #444

- Combine all customizable settings in one file. That way, there is only one place the user needs to remember to visit to make changes. This also saves us from having to write instructions like "to change setting A, go to file X, to change setting B, go to file Y." Keep it simple.

- Load the settings file once at runtime and assign the contents of the YAML to a constant.

- Document all customizable options thoroughly in settings.yml

- Take advantage of YAML keys that correspond to Rails environments to be able to use different settings in the test environment without having to create a new file and use multiple conditionals in helpers.

- Move hardcoded content to the settings file. For example, the SMC logo was replaced by the Ohana Web Search logo, but that should have been made a customizable option. Same with the default recipients of the feedback form and the site title.

- Make feedback mailer more customizable by using values from settings file to define the recipients and email subject.

- Fix CSRF Token bug in the feedback form. Rails 4 introduced protection against CSRF attacks, but when we upgraded, we failed to test the Feedback form, which uses AJAX to submit the form. The fix was to add the token to the headers.

- Move homepage links and info box helpers out of application_helper and into their own helper modules to keep functions separate and easier to read. Refactor methods and method names to make them smaller and more readable.

- Replace partials that use uninformative variables with more readable methods that render the HTML via content_tag.

- Rename partials to match helper names.

- Simplify info box lookup by only comparing keyword with synonyms field.

- Simply info box lookup by using "custom" key for info box that uses custom template.

- Use the Rails "try" method instead of multiple conditionals (http://apidock.com/rails/Object/try)

- Document all new methods.

- S... (continued)

48 of 48 new or added lines in 4 files covered. (100.0%)

241 of 267 relevant lines covered (90.26%)

56.7 hits per line

Relevant lines Covered
267 RELEVANT LINES 241 COVERED LINES
56.7 HITS PER LINE
Source Files on 336-remove-hardcoded-smc-code
Coverage ∆ File Lines Relevant Covered Missed Hits/Line

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
1367 336-remove-hardcoded-smc-code Improve code, fix bugs, and updates specs in PR #444 - Combine all customizable settings in one file. That way, there is only one place the user needs to remember to visit to make changes. This also saves us from having to write instructions like... push monfresh travis-ci pending completion  
1365 336-remove-hardcoded-smc-code Improve code, fix bugs, and updates specs in PR #444 - Combine all customizable settings in one file. That way, there is only one place the user needs to remember to visit to make changes. This also saves us from having to write instructions like... push monfresh travis-ci pending completion  
1363 336-remove-hardcoded-smc-code Fixes #336 - removes hardcoded SMC code Removes code specific to San Mateo County, CA. push anselmbradford travis-ci pending completion  
See All Builds (1890)