Skip to content
This repository has been archived by the owner on Aug 27, 2022. It is now read-only.

Пустые поля в занятиях #27

Open
VitalDikov opened this issue Mar 7, 2021 · 10 comments
Open

Пустые поля в занятиях #27

VitalDikov opened this issue Mar 7, 2021 · 10 comments
Labels
task New feature or request

Comments

@VitalDikov
Copy link
Contributor

VitalDikov commented Mar 7, 2021

У неоторых занятий нет Аудитории и Корпуса (у дистанционных например), а также нет Преподавателя. Это надо учитывать при создании верификатора, потому что потенциально может вызвать кучу проблем.

@VitalDikov
Copy link
Contributor Author

VitalDikov commented Mar 7, 2021

Если у двух предметов Room = null это не значит что они происходят в одной комнате

@s4xack
Copy link
Member

s4xack commented Mar 8, 2021

Это надо учитывать при создании верификатора

не нужно
это проблемы выгрузки из ису
при создании расписания таких ситуаций не должно быть

@FrediKats
Copy link
Member

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

@FrediKats FrediKats added this to the poc milestone Mar 8, 2021
@FrediKats FrediKats added the task New feature or request label Mar 8, 2021
@VitalDikov
Copy link
Contributor Author

при создании расписания таких ситуаций не должно быть

Я бы не сказал. При создании расписания можно создать занятие без препода. И тогда поле Teacher будет содержать null/default

@annchous
Copy link
Member

У неоторых занятий нет Аудитории и Корпуса (у дистанционных например)

а для дистанционных не будет разве написано online? по идее по аудиториям надо верифицировать только оффлайн занятия

@VitalDikov
Copy link
Contributor Author

а для дистанционных не будет разве написано online? по идее по аудиториям надо верифицировать только оффлайн занятия

тут дело даже не в онлайн занятиях. Англ у всех разный => у всех разные преподы/аудитории

@annchous
Copy link
Member

тут дело даже не в онлайн занятиях. Англ у всех разный => у всех разные преподы/аудитории

мы вроде для факультетских дисциплин пока что делаем инструмент
слабо представляю, как англичане будут этим пользоваться, да и кронбарсы тоже
а просто для преподов null вроде пойдёт

@VitalDikov
Copy link
Contributor Author

Ну то что онлайн занятия надо хендлить отдельно это факт, да.
Особой проблемы в нулл преподе (как и аудитории) нету. Просто надо будет с этим запариться на этапе верификатора, который будет не скоро

@s4xack
Copy link
Member

s4xack commented Mar 14, 2021

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

@annchous
Copy link
Member

предлагаю сделать условного пустого препода/аудиторию, который будет юзатьсяпри отстуствии информации
препод заглушка(вопыш)

на самом деле тоже об этом думала

@FrediKats FrediKats removed this from the poc milestone Nov 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
task New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants