Phil Pirozhkov
Phil Pirozhkov
Some more context https://github.com/rspec/rspec-rails/issues/2598#issuecomment-1109014190 - I withdraw my statement that adding `joinable: false` is a good practice. With `requires_new: true` and all explicit transactions (apart from implicit update/create/etc transactions), `joinable:...
Related cop suggestion https://github.com/rubocop/rubocop-rails/issues/400
Should it be possible to wrap a call to a gem that doesn’t `require_new` in a transaction to isolate it instead of a wrapping `joinable: false`? It is worth mentioning...
@koic @andyw8 @bbatsov WDYT?
This is so since Rails 3.0. Wondering why among ~200 usages of a-n-a, only two projects used this option? 🤔 Don't we usually pass the id along? By the way,...
https://github.com/rubocop/rails-style-guide/pull/295#issuecomment-1699388210 I’m on the fence. Let’s see real-workd usages to decide
Наверное, уже имеет смысл смотреть в сторону spree 1.1, который уже и выйдет на днях и доступен к использованию через ``` gem 'spree', :git => "git://github.com/spree/spree.git" ``` ```
Послал pull https://github.com/secoint/synergy/pull/24 Некоторое количество кода в настройках Engine закомментил, как то `Spree::Config.set(:allow_ssl_in_production => false)` и некоторые не очень понятные мне вещи. Заапдейтил README
Gemfile в студию. Обычная установка: `rails new myshop` Добавить в Gemfile synergy/spree: `gem 'spree', '0.7.0'` Запустить `bundle install rails g spree:install` Естественно, версия рельс должна соответствовать требуемой для используемой spree/synergy.