3fb085b3d9
* Adding more context to `/admin/abtests` Prior to this commit, the `/admin/abtests` was fully rendered by the field_test gem (see https://github.com/ankane/field_test/blob/master/app/views/field_test/experiments/index.html.erb and https://github.com/ankane/field_test/blob/master/app/views/field_test/experiments/show.html.erb). With this commit, we're pre-pending our own partial (e.g. `app/views/field_test/experiments/_experiments.html.erb`) in the view paths; which means instead of rendering https://github.com/ankane/field_test/blob/master/app/views/field_test/experiments/_experiments.html.erb we render our newly created `app/views/field_test/experiments/_experiments.html.erb`) Why the ugly antics in the view? As I'm not fully certain if this will "meet" the full needs of those monitoring the experiments. I'm also constructing this to most closely match the spreadsheet that has been assembled for tracking this information. Why introduce another caching layer for the variant assembly? The original caching layer (e.g. `Articles::Feeds::VariantAssembler.pre_assembled_variants`) is in place to provide the quickest access to the assembled variants. But due to the nature of the implemented view, we are needing a programmatic representation of all variants. And the added Rails cache is there to minimize disk reads. Closes forem/forem#17820 * Update app/models/articles/feeds/variant_assembler.rb Co-authored-by: Ridhwana <Ridhwana.Khan16@gmail.com> * Update app/models/articles/feeds/variant_assembler.rb Co-authored-by: Ridhwana <Ridhwana.Khan16@gmail.com> Co-authored-by: Ridhwana <Ridhwana.Khan16@gmail.com> |
||
|---|---|---|
| .. | ||
| feeds | ||
| cached_entity_spec.rb | ||
| feeds_spec.rb | ||