No need to define to_param in Product again

friendly_id already takes care of it
Set association_foreign_key for spree_promotion_rules_users

Probably because core default user class is LegacyUser rails would guess
the column as legacy_user_id. Green master build again