Mykolai Sonych
sonych

Aug 18, 2016
@sonych
  • @sonych 6368c90
    PY-2016.1.4 <sonych@sonych-mbp.dev.uaprom Update packages.xml
Aug 18, 2016
@sonych
  • @sonych cbe0133
    PY-2016.1.4 <sonych@sonych-mbp.dev.uaprom Update packages.xml
Aug 15, 2016
@sonych
  • @sonych 3e5260f
    PY-2016.1.4 <sonych@sonych-mbp.dev.uaprom Update packages.xml
Aug 11, 2016
@sonych
  • @sonych 2b18349
    PY-2016.1.4 <sonych@sonych-mbp.dev.uaprom Update packages.xml
Aug 10, 2016
sonych commented on pull request rkulak/banner-creator#41
@sonych

третий раз вижу :) пишите декоратор который будет проверять наличия поля file в теле запроса

Aug 10, 2016
sonych commented on pull request rkulak/banner-creator#41
@sonych

Формирование имени файла тоже б вынести в отдельную функцию. Что бы если изменился принцип формирования имени, то нужно было изменить только одну ф…

Aug 10, 2016
sonych commented on pull request rkulak/banner-creator#41
@sonych

блок else тут и в make_review очень похожи. выделите общую часть (декодирование изображения, создание файла, генерирование имени) в внутреннюю функцию

Aug 10, 2016
sonych commented on pull request rkulak/banner-creator#41
@sonych

если еще третий раз есть такая проверка if 'file' not in request.json: то напишите декоратор который будет проверить наличие file в теле запроса

Aug 10, 2016
sonych commented on pull request rkulak/banner-creator#41
@sonych

снова, если дизайнеров не будет в БД шаблон там не упадет?

Aug 10, 2016
sonych commented on pull request rkulak/banner-creator#41
@sonych

4 – это ж количество элементов на странице? в идеале такое нужно в конфиг вынести, ну или на крайняк хоть константой сделать

Aug 10, 2016
sonych commented on pull request rkulak/banner-creator#41
@sonych

шаблон нормально себя поведет если выборка будет пустая?

Aug 10, 2016
sonych commented on pull request rkulak/banner-creator#41
@sonych

это модельно окно еще будет возвращаться? если нет – удалить закоментированный код

Aug 10, 2016
sonych commented on pull request rkulak/banner-creator#41
@sonych

Проверка роли это больше бизнес логика чем отображение. Лучше создайте отдельную функцию, котороя будет принимать пользователя как аргумент и пров…

Aug 10, 2016
sonych commented on pull request rkulak/banner-creator#41
@sonych

onClick это не очень практика. обсудите с кем-то кто хорошо шарит в фронтенде как от него избавится

Aug 10, 2016
sonych commented on pull request rkulak/banner-creator#41
@sonych

ай, ну переименуйте эти два ключи из snake_case в camelCase

Aug 10, 2016
@sonych
Aug 9, 2016
@sonych
  • @sonych 9f2bc29
    PY-2016.1.4 <sonych@sonych-mbp.dev.uaprom Update find.xml
Aug 9, 2016
@sonych
Aug 5, 2016
@sonych
  • @sonych 42f4b3b
    PY-2016.1.4 <sonych@sonych-mbp.dev.uaprom Update packages.xml
Aug 4, 2016
sonych commented on pull request rkulak/banner-creator#34
@sonych

не проверил что поле designer есть в форме. вы не думали взять либу для валидации форма? например вот неплохая http://flask.pocoo.org/docs/0.11/pat…

Aug 4, 2016
sonych commented on pull request rkulak/banner-creator#34
@sonych

если сохранил в форму в переменную form, то и пользуйся этой переменной. а то вдруг ключ file будет в request.form, но не будет в form. (в данном с…

Aug 4, 2016
sonych commented on pull request rkulak/banner-creator#34
@sonych

и здесь пропустил проверку 'id' в форме

Aug 4, 2016
sonych commented on pull request rkulak/banner-creator#34
@sonych

а почему это ресурс позволяет использовать GET? вюха как я вижу заточена на работу с POST

Aug 4, 2016
sonych commented on pull request rkulak/banner-creator#34
@sonych

и что бы вложенность кода была меньше здесь лучше без секции else @login_required def save_cuted(): if 'file' not in request.json or 'name' not in …

Aug 4, 2016
sonych commented on pull request rkulak/banner-creator#34
@sonych

а наличие 'name' не проверил

Aug 4, 2016
sonych commented on pull request rkulak/banner-creator#34
@sonych

про 406 вики говорит: 406 Not Acceptable — запрошенный URI не может удовлетворить переданным в заголовке характеристикам здесь более подходящий код…

Aug 4, 2016
sonych commented on pull request rkulak/banner-creator#34
@sonych

убрать консоль лог

Aug 4, 2016
sonych commented on pull request rkulak/banner-creator#34
@sonych

убери console.log

Aug 3, 2016
@sonych
  • @sonych b522243
    PY-2016.1.4 <sonych@sonych-mbp.dev.uaprom Update packages.xml
Aug 3, 2016
sonych commented on pull request rkulak/banner-creator#30
@sonych

file – это имя встроенной функции. от того что ты его перезаписал своим объектом нет, пока ты не попытаешься в этой области видимости эту функцию и…