ohmycto / synergy

Решение для создания российских интернет-магазинов, основанное на Spree.
GNU General Public License v3.0
93 stars 45 forks source link

Заставил работать с 1.1.0 #24

Closed pirj closed 6 years ago

pirj commented 12 years ago

Предлагаю доработать это совместно. Кое-что я закомментил, в особенности отключение HTTPS в продакшне и др. Валидации preferences пришлось выключить, потому как в 1.1 это не работает. Задал вопрос в гуглогруппах. Ответ - использовать костыли.

Было бы неплохо разбить это всё на модули, один для работы с юрлицами, один для квитанции сбера, а собственно synergy уже чтобы собирать всё нужное в кучу, а тому, кому это не нужно (мне, например) не нужно было включать всякие spree_static_content итп. spree_address_book не работает, и смысла в нём не вижу, так как сейчас Spree и сам подставляет ранее забитый адрес. spree_address_book не предлагал выбора из списка или добавление новых адресов, как я понял, что не хорошо.

ohmycto commented 12 years ago

Немного с проектами разгребемся и займемся этим обязательно!

GarPit commented 12 years ago

Когда-это будет смержено?

pirj commented 12 years ago

Вот тут, кстати, интересно по поводу spree_default_theme и spree_address_book https://github.com/syndicut/synergy/commit/1862d191c7a9b076e3e1919d929abf42d1536ac5

pirj commented 12 years ago

Работает стабильно с 1.1.3. К новому году думаю прикрутить к 1.2 или 1.3, что там стабильнее будет. Если не смерджите, то поставьте хотя бы ссылку на мой форк в ридми, что типа мы только 0.5, 0.6 и с грехом пополам 0.7 поддерживаем, а за новыми стучитесь туда-то.

ohmycto commented 12 years ago

Добрый день, спасибо за Ваш вклад в развитие проекта! На этой неделе мы запланировали сделать мердж вашей ветки.

pirj commented 12 years ago

Люди спрашивают, нет ли тестов. Для secondname нужно проставить attr_accessible, чтобы защита на massprotection не срабатывала. С оплатами надо проверить, я делал только распечатку счёта для юрлиц. Квитанция сбера не полностью заполняется. Ещё неплохо смерджить с веткой syndicut/synergy, там приделана поддержка address_book. По админке нужно слегка доделать i18n. Хотя, возможно, что в основном для плагинов.

pirj commented 12 years ago

Вот, кстати, ряд тулз для тестирования rails engines https://www.ruby-toolbox.com/categories/Testing_Rails_Engines

impedance commented 11 years ago

Добрый день, откуда можно взять вашу ветку "Работает стабильно с 1.1.3. К новому году думаю прикрутить к 1.2 или 1.3"?

pirj commented 11 years ago

@impedance https://github.com/pirj/synergy А тут кое-какие инструкции по применению. https://groups.google.com/forum/?fromgroups=#!searchin/ror2ru/spree/ror2ru/AWQ85KkxXCI/5Ox2P8-gElkJ