Stopień studiów studenta określany jest tylko za pomocą ciągu znaków wpisywanego przez administratora zapisów. W modelu Program powinny znaleźć się odpowiednie pola określające stopień studiów (licencjackie/inżynierskie/magisterskie/doktoranckie) i typ (wieczorowe/dzienne/jednolite), etc.

przeklejone z bugs

I think it might be useful to add a close-blocking notification there because it's really easy to close it when you have dozens other tabs opened and they all look the same. (Yeah, it just happened to me when I was almost ready to save 😿)

Zamiast wysyłania maili do wszystkich studentów poprzez mailto, możnaby mieć feature do dystrubucji maili, np. wysyłając na adres [email protected] wiadomość dochodzi do wszystkich zapisanych do grupy. Plus jest taki, że prowadzący nie musi za każdym razem aktualizować sobie listy maili do studentów. Drugi plus - gdybyśmy mieli listę ze wszystkimi aktywnymi studentami (aktualizowaną automatycznie) dyrekcja mogłaby wysyłać wiadomości bezpośrednio na tą listę.

Anonymous created an issue 2015-09-29
Miejsce problemu: Logika dla informatyków
Dostępne wykłady: 2
Skutki:

1. Zapis na dowolne ćwiczenia powoduje zapisanie na wszystkie dostępne wykłady (wykłady są w tym samym czasie, w innej sali, na innym poziomie nauczania).
2. Wypisanie z jednego wykładu powoduje wypisanie z przedmiotu

5 stycznia (czwartek) odbędą się zajęcia piątkowe (w SZ służy do tego fereol_admin/courses/changedday/)

Przy rezerwacji sali tego dnia formularz słusznie pokazuje, że sala 25 jest wolna między 12 a 14 (bo w piątki nie ma tam zajęć).

Jednak gdy klika się zapisz pojawia się informacja, że jest wtedy wykład z analizy matematycznej (wykład jest, ale w czwartki). Dni zamienione na inne nie zadziałały w tej drugiej walidacji.

Screenshot:

Można dodać do naszego CI mierzenie test coverage

W tej chwili nie mamy widoku dla pojedynczego newsa, więc w mailu z powiadomieniem o nowym newsie wysyłamy link do wszystkich newsów. Bardziej intuicyjne byłoby wysyłanie linku do nowo dodanego newsu.

Należy dodać kod sprawdzający czy po zmianie limitów nie trzeba wciągnąc studentów z kolejki. Najlepszą lokalizacją będzie prawdopodobnie metoda save() obiektu Group.

Add the notification that website uses cookies.

• Read http://rogueleaderr.com/post/65157477648/the-idiomatic-guide-to-deploying-django-in
• Make chefable environment
• Add monitoring, for example: new relic
• Encryption of secrets
• Add useful tools
  • htop
• Clean up the settings.py, make it versioned in a system
• Add error tracking software: rollbar looks like the one which is free and supports python2.6
  • It works! It just needs an upgrade of six to 1.9.0
• Clean directories which should be linked on production/staging systems (logs/, at least)
• Add logs with logrotate [ logs are rotated, but not with the logrotate. Setup was annoying as fuck, it may be easier to do it right when on newer versions of Django and/or python ]
• CircleCI setup
  • Notifications to slack
  • Github PR CI badges
• Make deployment with switchable releases
• Queue tools (Celery, RabbitMQ) [ I would postpone until we'll be on python >= 2.7]
• Backups
• NewRelic monitoring for Django application
• Accounts
  • new relic
  • rollbar
• Find better domain than my personal zapisyuwr.swistak35.com for staging
• Make deploy install packages
• Make deploy run migrations

Sugestia: Interfejs byłby przyjaźniejszy, gdyby lista prowadzących była posortowana.

j/w

Przy dodawaniu cyklicznych rezerwacji powinniśmy mieć wybór jaki typ mają te zajęcia (CLASS, EVENT, etc.)

Potrzebujemy tutaj założyć dwa konta, na jakiś adres e-mail systemu zapisów:

• newrelic.com
• rollbar.com

Aktualnie obie rzeczy robiłem na prywatnych kontach.

Not accepting a course proposition would result in putting it on the offer list as normal course
compare get_proposals in zapisy/apps/enrollment/courses/models/course.py and zapisy/templates/common/coursesitebar.html (for review code is 5).

W zeszłym semestrze były problemy z dostępem, niektórzy użytkownicy mieli dostęp do widoków, których nie powinni widzieć

Zadania:

• Opisać sposób w jaki system sprawdza dostęp użytkownika do widoku i jesli zajedzie taka potrzeba poprawić go
• Upewnić się, że wszystkie widoki które powinny być dostępne tylko dla pracowników są niedostępne dla studentów

Łukasz Arczyński created an issue 2016-10-13
Na stronie https://zapisy.ii.uni.wroc.pl/vote/view jest link do głosowania, który zwraca 404, gdy głosowanie jest nieaktywne

Anonymous created an issue 2016-10-04
Jako student gdy jestem zapisany do kolejki na ćwiczenia/pracownie oraz znajduję się w kolejce na wykład w momencie, gdy zwolni się miejsce na ćwiczeniach ( na wykładzie dalej znajduje się w kolejce). To automatycznie zostaję zapisany na wykład ale także nadal znajduję się na liście oczekujących co skutkuje zdublowaniem wpisu zapisani/w kolejce.
Przykład występowanie błędu: Kurs: Wybrane elementy praktyki projektowania oprogramowania (2016/17 zimowy) https://zapisy.ii.uni.wroc.pl/records/13588/records

Autor i opiekuni SZ powini mieć możliwość usunięcia wydarzenia.

Zawsze po ustawieniu środowiska trzeba ręcznie zmieniać hasła użytkowników, z których chcemy testować aplikacje. Możnaby dodać pare poleceń do naszych skryptów które ułatwiłyby rozpoczęcie pracy.

To Do:

• Remove all migrations
• Remove all tests that are red or non-sense

@florczakraf na slacku:

Jakiś hardcode semestru w zapytaniu (za jakiś czas przyjdzie pytanie z góry: "A dlaczego są cały czas stare statystyki?"):
https://github.com/lewymati/projektzapisy/blob/master/zapisy/apps/statistics/views.py#L21-L29

Odwiedzenie /statics/swap powoduje wykonanie sporej liczby zapytań i bład http 500. Przykład błędu:

NoReverseMatch at /statistics/swap/
Reverse for ''records-group'' with arguments '(13297,)' and keyword arguments '{}' not found.

Gdy sala jest zarezerowana do godziny 15:00, nie można jej zarezerwować od tej godziny. Podobnie jest z drugim końcem przedziału. Przykład - sala 25 poniżej.

Np. /courses/algebra_1516

(Studenci w grupach: , studenci bez grupy: )

• zrozumieć widok course z apps/enrollment/courses/views.py
• w miarę możliwości pozbyć się SQLa na rzecz zapytań django
• naprawić błąd
• przenieść większość kodu z widoku do odpowiednich metod modeli

12d3be2#commitcomment-19104871

Prawdopodobnie wystarczy żeby dodać jakiegoś SORT po tym record_closing i wybrać najwcześniejszy

Description

There are still some (known) tasks to do to complete upgrade to Django 1.5.
If you think that there's something else to do, feel free to update this issue.
Any help is welcome as well.

Tasks

• Update url syntax in templates
• Fix usage of static files in templates
• Fix static files in rest of the project (o___O for example: https://github.com/lewymati/projektzapisy/blob/master/zapisy/apps/news/admin.py)
• Update manage.py (because there's already warning about it's deprecation)
• Figure out how to properly query if user is student or employee
• There are some other deprecation warnings that have to be eliminated

nie można akceptować zwykłych rezerwacji. Jak kilkam decyzja->zaakceptowano (albo nawet Odrzucono) to wyświetla mi info: 'Status wydarzenia nie został zmieniony'.

Prawdopodobnie ta linia powoduje błąd: https://github.com/lewymati/projektzapisy/blob/master/zapisy/apps/schedule/views.py#L129

z jakiegoś powodu event.status i form.cleaned_data['status'] mają zawsze taką samą wartość (1 lub 2 w zależności co kilknę by zrobić)

Prawie wszystkie modele powinny mieć przeciążone metody clean()

Wg. aplikacji:

• enrollment/courses
• enrollment/records
• offer
• schedule
• users

Przydałoby się najpierw napisać testy.

schedule/reservation: Przy rezerwacji jest problem z rezerwacją sali w której właśnie kończą się zajęcia, tj. jeśli sala jest zajęta do 12:15 to powinno dać się ją zarezerwować od 12:15. Błąd dotyczy formularza.

Opisy przedmiotów powinny być posortowane według daty.

Przykład: /fereol_admin/courses/course/3514/

Moim zdaniem relacja teachers z modelu Course powinna zostać usunięta.
Ta sama informacja przechowywana jest w Group.teacher, można się do niej dostać wykonując tą samą liczbę zapytań, na tabeli o tym samym rozmiarze.

Filtr Kursy i seminaria nie działa na K1 i K2 -- a co za tym idzie nie ukrywa/pokazuje kursów na liście.

A tak na boku, to nie wiem czy nie przydałby się jakiś rework też tych filtrów z prawej (pokaż/schowaj) -- chodzi mi o ich ujednolicenie.

Description

Our server runs on the II's VPS. Machine has whole environment setup in bare-bone style. It needs some more attention.

Right now everything runs on unattached server process running on the machine. It is not really secure and requires work on automating deploys.

Good thing to think about - do we really need to automate this process? I am not that sure... IMO, just git pulling by hand is enough.

Run server

1. Enter [email protected]
2. Get into var/lib/projektzapisy/zapisy and run nohup python manage.py runserver 0.0.0.0:8000 &
3. Immediately leave terminal session. Sometimes control may get back to your user

TO DO

Server infrastructure

• Automate running Django process with proper env
• Enforce Basic Auth authorization, to restrict access to whole website
• Ensure, you cannot connect from outside the machine to postgres

CI

• Setup Jenkins
  • Setup Testing running
  • Setup deploys after commit

W projekcie jest bardzo dużo "magicznych stałych", np:

def is_proposal(self):
    return (self.status == 0) or (self.status == 5)

Powinniśmy zamienić je na stałe zdefiniowane w odpowiednich klasach.

Kalendarz nie pokazuje wszystkich zajęć, które odbywają się w wybranej sali.

W adminie można ustawiać następujące dodatkowe limity:
Miejsca dla zamawianych 2009 Miejsca dla zamawianych 2012 Miejsca dla zamawianych 2012

Te drugie 'Miejsca dla zamawianych 2012' odpowiadają za ISIM. Trzeba zmienić nazwę.

przeklejone z bugs

Jestem na masterze

(env2.6) vagrant@vagrant-ubuntu-trusty-64:/var/www/zapisy$ python manage.py run
server 0.0.0.0:8000
/home/vagrant/env2.6/lib/python2.6/site-packages/django/conf/__init__.py:75: Dep
recationWarning: The ADMIN_MEDIA_PREFIX setting has been removed; use STATIC_URL
 instead.
  "use STATIC_URL instead.", DeprecationWarning)
Traceback (most recent call last):
  File "manage.py", line 14, in <module>
    from django.contrib.auth.models import Permission
  File "/home/vagrant/env2.6/lib/python2.6/site-packages/django/contrib/auth/mod
els.py", line 5, in <module>
    from django.db import models
  File "/home/vagrant/env2.6/lib/python2.6/site-packages/django/db/__init__.py",
 line 11, in <module>
    if DEFAULT_DB_ALIAS not in settings.DATABASES:
  File "/home/vagrant/env2.6/lib/python2.6/site-packages/django/utils/functional
.py", line 184, in inner
    self._setup()
  File "/home/vagrant/env2.6/lib/python2.6/site-packages/django/conf/__init__.py
", line 42, in _setup
    self._wrapped = Settings(settings_module)
  File "/home/vagrant/env2.6/lib/python2.6/site-packages/django/conf/__init__.py
", line 135, in __init__
    logging_config_func(self.LOGGING)
  File "/home/vagrant/env2.6/lib/python2.6/site-packages/django/utils/dictconfig
.py", line 553, in dictConfig
    dictConfigClass(config).configure()
  File "/home/vagrant/env2.6/lib/python2.6/site-packages/django/utils/dictconfig
.py", line 352, in configure
    '%r: %s' % (name, e))
ValueError: Unable to configure handler 'logfile': [Errno 2] No such file or dir
ectory: '/vagrant/zapisy/logs/djangoproject.log'
(env2.6) vagrant@vagrant-ubuntu-trusty-64:/var/www/zapisy$

W env setup trzeba dodać:

mkdir /vagrant/zapisy/logs
touch /vagrant/zapisy/logs/djangoproject.log

Brakowało też rollbara - pomogło pip install rollbar

Zarządzanie listą dni wolnych od zajęć jest uciążliwe. Gdyby były posortowane według daty byłoby to sporym ułatwieniem.

Screenshot jak jest teraz:

Dodawanie cyklicznej rezerwacji daje 500tkę jeśli wykryty jest konflikt z wydarzeniem, które ma 'ł' w nazwie:

2016-09-27 02:44:03,117 [MainThread  ] [ERROR] [django.request      ]  Internal Server Error: /fereol_admin/schedule/specialreservation/add/
Traceback (most recent call last):
  File "/home/zapisy/env26/lib/python2.6/site-packages/django/core/handlers/base.py", line 111, in get_response
    response = callback(request, *callback_args, **callback_kwargs)
  File "/home/zapisy/env26/lib/python2.6/site-packages/newrelic-2.68.0.50/newrelic/hooks/framework_django.py", line 503, in wrapper
    return wrapped(*args, **kwargs)
  File "/home/zapisy/env26/lib/python2.6/site-packages/django/contrib/admin/options.py", line 366, in wrapper
    return self.admin_site.admin_view(view)(*args, **kwargs)
  File "/home/zapisy/env26/lib/python2.6/site-packages/django/utils/decorators.py", line 91, in _wrapped_view
    response = view_func(request, *args, **kwargs)
  File "/home/zapisy/env26/lib/python2.6/site-packages/django/views/decorators/cache.py", line 89, in _wrapped_view_func
    response = view_func(request, *args, **kwargs)
  File "/home/zapisy/env26/lib/python2.6/site-packages/django/contrib/admin/sites.py", line 196, in inner
    return view(request, *args, **kwargs)
  File "/home/zapisy/env26/lib/python2.6/site-packages/django/utils/decorators.py", line 25, in _wrapper
    return bound_func(*args, **kwargs)
  File "/home/zapisy/env26/lib/python2.6/site-packages/django/utils/decorators.py", line 91, in _wrapped_view
    response = view_func(request, *args, **kwargs)
  File "/home/zapisy/env26/lib/python2.6/site-packages/django/utils/decorators.py", line 21, in bound_func
    return func(self, *args2, **kwargs2)
  File "/home/zapisy/env26/lib/python2.6/site-packages/django/db/transaction.py", line 209, in inner
    return func(*args, **kwargs)
  File "/home/zapisy/env26/lib/python2.6/site-packages/django/contrib/admin/options.py", line 937, in add_view
    if form.is_valid():
  File "/home/zapisy/env26/lib/python2.6/site-packages/django/forms/forms.py", line 124, in is_valid
    return self.is_bound and not bool(self.errors)
  File "/home/zapisy/env26/lib/python2.6/site-packages/django/forms/forms.py", line 115, in _get_errors
    self.full_clean()
  File "/home/zapisy/env26/lib/python2.6/site-packages/django/forms/forms.py", line 272, in full_clean
    self._post_clean()
  File "/home/zapisy/env26/lib/python2.6/site-packages/django/forms/models.py", line 332, in _post_clean
    self.instance.clean()
  File "/home/zapisy/projektzapisy/releases/1474674344/zapisy/apps/schedule/models/specialreservation.py", line 144, in clean
    self.validate_against_event_terms()
  File "/home/zapisy/projektzapisy/releases/1474674344/zapisy/apps/schedule/models/specialreservation.py", line 123, in validate_against_event_terms
    unicode(terms[0].event) + ' ' + unicode(terms[0])]},
  File "/home/zapisy/projektzapisy/releases/1474674344/zapisy/apps/schedule/models/event.py", line 262, in __unicode__
    smart_unicode(self.description))
UnicodeEncodeError: 'ascii' codec can't encode character u'\u0142' in position 52: ordinal not in range(128)

Klikając 'wyślij wiadomość do grupy' uruchamia się klient pocztowy z wpisanymi adresami jako odbiorcy.
Lepiej będzie jak odbiorcy wpiszą się automatycznie do bcc.

Jeśli użytkownik jest niezalogowany to na www.zapisy.ii.uni.wroc.pl/offer nie powinien w ogóle móc widzieć 'wyszarzonych' przedmiotów (wycofanych z oferty)

Pole: "Wydarzenie widoczne dla wszystkich użytkowników systemu" powinno być domyślnie zaznaczone (/classrooms/reservation)
Na prośbę ASm.

Issue do wykonania w 5 minut ;)

Przeliczanie punktów przy głosowaniu nie działa do końca dobrze. Przykład z głównego głosowania na rok 2016/2017, przykładowo przy wybieraniu przedmiotu "Charakterystyka interakcji społecznych", liczba punktów na górze się nie zmienia.

https://projektzapisy.slack.com/archives/bugs/p1465896894000002

W ustawieniach głosowania w panelu administratora (fereol_admin/vote/systemstate/) pole początek/koniec korekty jest niepotrzebne (duplikuje początek/koniec korekty zimowej). Trzeba to usunąć.

Nie działa jak w #34 + dodatkowo nie działa ukrywanie przedmiotów, na które już się kiedyś chodziło (checkbox).

When writing mails to group or queue, we don't want to use bcc always.

Ze względu na tę linię kodu pracownicy nie mogą dodawać egazminów:

qs = qs.filter(teachers=user.employee)

w apps/schedule/forms.py.

PS: podobno jest tak od zawsze