Рубрики
Без рубрики

Полировка и удобство использования – здание Саас # 69

В этом эпизоде мы отслеживали некоторые части приложения. Теперь, когда мой первый клиент использует T … с меткой Python, Django, вид.

Здание Саас (86 части серии)

В этом эпизоде мы отслеживали некоторые части приложения. Теперь, когда мой первый клиент регулярно использует приложение, обратная связь быстро придет. Мы работали, чтобы исправить некоторые из вопросов, которые она нашла.

Первый вопрос, который я решал рассмотрел двусмысленность в отношении отношения курса с учебным годом на странице детализации курса. Я исправил эту проблему, отображая уровень класса на странице курса, чтобы обеспечить все детали. Это изменение делает его понятно, к чему подключено курс. Это полезно, потому что курсы в учебном году могут иметь то же имя (например, «математика»).

Следующий вопрос, на котором мы работали, было связано с упорядочением данных на страницах расписания. Страницы расписания показали информацию для каждого ученика в школе. Это было сломано на пару причин:

  1. Порядок студентов не соответствует порядку уровней класса. Это приводит к несоответствию при взгляде на учебный год по сравнению с взглядом на график. 2. Расписание будет отображаться студентам, даже если они не были зачислены в учебный год.

Я зафиксировал ежедневные и еженедельные представления расписания, изменив логику из петли по всем студентам, чтобы зацикливаться за всеми зачислениями на учебный год. Это изменение исправило обе проблемы, которые присутствовали в представлении.

После изменения этого в двух месте я наблюдал общую реализацию, которая была дублирована в два взглядах. Из-за дублирования мы сделали некоторую рефакторию для извлечения этой логики в общий метод на Студент модель.

В конце потока, поскольку я убрал код, чтобы подготовиться к его компетенту, мы убрали какой-то тестовый код на основе некоторых наблюдений о том, как тесты велят с новым кодом.

Здание Саас (86 части серии)

Оригинал: “https://dev.to/mblayman/polishing-and-usability-building-saas-69-1p6j”