1
0%
master: 0%

Ran 30 Mar 2017 09:11AM UTC

Files 36

Run time 1s

Badge

Embed ▾

Committed 30 Mar 2017 08:56AM UTC coverage: 97.906% (+0.008%) from 97.898%

Job # 2.2.6, spec/dummy/Gemfile

Build Type

push

travis-ci

Committed by

justin808

Commit Message

Use <script type="application/json"> for props and store (#775) for performance

Initial implementation started by https://github.com/eliaslopezgt and then https://github.com/squadette
* Refactor console output to use <script> tag with a certain ID
* Use <script type="application/json"> with JSON content instead of attribute
* Pretty-print JSON in development and testing
* getAttribute() is better, because it handles attributes which are not present.
* Return skip display none to react on rails configuration
* Extract data-js-react-on-rails-store attribute to the const
* Return to_json method call to app_props_server_render to ensure handling the both cases
* Delete unnecessary style method and HIDDEN constant from react component module
* Improve jsonEl parsing to handle props as already converted from JSON to a string
* Move handling props already converted to json string to ruby code
* Escape json from props as json string
* Add test to ensure that even json string props are sanitezed
* Delete test json props because we parse all props to a hash now

Run Details

748 of 764 relevant lines covered (97.91%)

41.31 hits per line

shakacode / react_on_rails / 3240 / 1
0%
master: 0%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 3240.1 (2.2.6, spec/dummy/Gemfile)

shakacode / react_on_rails / 3240 / 1 0% master: 0%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 3240.1 (2.2.6, spec/dummy/Gemfile)

shakacode / react_on_rails / 3240 / 1
0%
master: 0%

README BADGES
x