mkozhukharenko
commented
9 years ago fixed, please test
Open Krapka opened 9 years ago
mkozhukharenko
commented
9 years ago fixed, please test
Krapka
commented
9 years ago Повідомлення немає, але персона тепер не зберігається. Ну і нотифікейшн все ж має бути, якщо поля з адресою не заповнені
mkozhukharenko
commented
9 years ago прибрав напис "додайте адресу" але разом з тим і нотіфікейнш про який ти кажеш. Справав в тому что прогрес бар (сам прогрес бар, нотіфікейншини, дисейбл-енейбл кнопка зберегти яка прикручується до бару) - це все один модуль, який не працює корректно для жодної з вьюх (нова заявка, нова персона, новий спекофер). Виправивши якісь недоліки по прогрес бару тут покличуть за собою недоліки в інших вьюхах. Тому пропоную винести окремий великий task "реалізація програс бару для всіх view" і позначити її як enhancement, а на даному етапі позбутися явних багів, тому що прогрес бар не є критичною фічою. (явних багів вже немає).
Наразі поведінка реалізована наступним чином: напис на прогрес барі тепер: "Основна інформаці заповнена на: 100%" (тобто заповнена лише перша таба) і кнопка "зберегти" прикручена поки лише до події коли заповниться перша таба на 100%.
В майбутньому необхідно реалізувати щоб кнопка була disabled якщо і інші таби були не заповнені і показувалися відповідні повідомлення. Варто зауважити що в різних views різні таби тому потрібно кастомізувати повідомлення.
mkozhukharenko
commented
9 years ago цей таск має пряме відошення до таску https://github.com/ifnul/ums-admission-frontend/issues/95
При редагуванні існуючої персони, з'являється повідомлення "Будь ласка, додайте адресу!". При тому, що всі поля на кожній вкладці, заповнені