-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
hot fix #2
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hot fix
@megabogus а что ты хочешь в ветке мастер получить в итоге? зачем тебе эти правки в ней, не будет ли проблем с другими проектами? Где CHANGELOG ? |
Сейчас есть жесткая ошибка не позволяющая сохранить лид, сейчас ветка мастер не работает в плане отправки данных формы |
Exception Type: AttributeError at /feedback/new/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Нужно до конца доделать эту задачу с выключением медиацентра. ДОбавить документацию с примером интеграции на уровне приложения, и удалять из кода месте, а не комментировать или по возможности и если это идеологически верно делать места конфигурируемыми.
Например на уровне шаблонов можно делать так сразу, на уровне методов с сервисами тоже, а вот на уровне наследования класса от медиацентра не уверен что сработает конфигурацию и тут нужно писать документ с примером.
return api.register['media'][self].get_instances( | ||
consumer=self, is_public=True) | ||
return [] | ||
#return api.register['media'][self].get_instances( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Этот код я не буду принимать. Тут надо делать универсальную конструкцию с проверкой модуля spicy.mediacenter в приложении и если его нет то метод ничего не возвразает, а вы просто комментарии ставите. Так не пойдет.
|
||
def has_attachments(self): | ||
return bool(api.register['media'][self].get_instances( | ||
consumer=self, is_public=True).count()) | ||
return False |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
То же самое, халтура какая-то
@@ -2,11 +2,10 @@ | |||
from django.db import models | |||
from django.utils.translation import ugettext_lazy as _ | |||
from spicy.core.service import models as service_models | |||
from spicy.mediacenter.abs import FatMediaConsumerModel |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Тут сложный момент с подклчение этого модуля. вот импорт мы уже условно проверить не сможем в зависимоти от конфигурации и тогда получается, что подключение загрузки файлов можно сделать только на уровне приложения. В тако м случае это должно быть указанно в документации модуля лдя программиста и дан пример конфигурации, а тут в коде просто удалены все эти места с комемнтариями. А сейчас получается что вы закомментили куски, отключили а как включить никому не сказали. Хотите чтобы к-то то раскомментировал код?
Делайте нормально до конца. Этот код я не приму.
@@ -6,6 +6,6 @@ | |||
{% for backend in backends %} | |||
<li{% if tab == backend.1 %} class="active"{% endif %}><a href="{% url 'feedback:admin:edit-pattern' form.instance.pk backend.1 %}"><i class="icon-file-alt"></i> <span>{{ backend.0 }}</span></a></li> | |||
{% endfor %} | |||
<li{% if tab == 'media' %} class="active"{% endif %}><a href="{% url 'feedback:admin:edit-pattern-media' form.instance.pk %}" ><i class="icon-camera"></i> <span>{% trans "Attachments" %}</span></a></li> | |||
{#<li{% if tab == 'media' %} class="active"{% endif %}><a href="{% url 'feedback:admin:edit-pattern-media' form.instance.pk %}" ><i class="icon-camera"></i> <span>{% trans "Attachments" %}</span></a></li>#} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Зачем тут опять коментарий?
@@ -15,8 +15,8 @@ | |||
url( | |||
r'^patterns/(?P<pattern_id>\d+)/extra/(?P<backend_name>\w+)/$', | |||
'edit_pattern', name='edit-pattern'), | |||
url(r'^patterns/(?P<pattern_id>\d+)/attach/$', 'pattern_media', | |||
name='edit-pattern-media'), | |||
#url(r'^patterns/(?P<pattern_id>\d+)/attach/$', 'pattern_media', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
жесть
url( | ||
r'^patterns/attach/(?P<pattern_id>\d+)/$', 'pattern_media', | ||
name='edit-pattern-media'), | ||
#url( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Жесть
@@ -45,8 +45,6 @@ def new_feedback(request): | |||
feedback.processing_status = defaults.SPAM | |||
feedback.save() | |||
|
|||
resp = pattern.save_in_api(request) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
А что это было?
No description provided.