Skip to content
This repository was archived by the owner on Nov 23, 2024. It is now read-only.

Comments

add email domain to student index#32

Open
StachowiakDawid wants to merge 1 commit intomasterfrom
#5
Open

add email domain to student index#32
StachowiakDawid wants to merge 1 commit intomasterfrom
#5

Conversation

@StachowiakDawid
Copy link
Contributor

No description provided.

@karolba
Copy link
Contributor

karolba commented Aug 2, 2022

Podlinkuję ponieważ nie ma, ten pull request ma komentarze w jego dwóch starszych porzuconych wersjach: #26 i #29

@karolba
Copy link
Contributor

karolba commented Aug 8, 2022

Odpowiedź do #26 (comment)

Dodatkowo dodał bym check czy tekst do którego dopisujemy @student.pwr.edu.pl rzeczywiście jest indeksem.

Trzeba też się zastanowić co zrobić z tekstem który nie jest indeksem i też nie jest mailem - czyli ewidentną pomyłką. Najlepiej było by jakoś powiadomić użytkownika że popełnił błąd, ale w taki sposób żeby nie był przytłoczony dużą ilością błędów w przypadku dużej ilości błędów.

#27 (comment)

Rozumiem że chcesz wprowadzić frameworka na frontend dla wyświetlenia textarea z adresami email na frontend jako jakiejś interaktywnej tabelki dla pokazywania błędów, tak? Ale co byś powiedział na prostszą komunikację z użytkownikiem o błędach - według mnie wyświetlenie po prostu listy błędnych danych nad listą maili (podobny modal jak przy błędach) było by i lepsze dla użytkownika i proste do zaimplementowania.

@karolba
Copy link
Contributor

karolba commented Aug 8, 2022

Odpowiedź do #26 (comment)

Dodaj też opis o tym że można wklejać indeksy zamiast adresów do tej linijki, tak samo w editpoll.html

Dodany

Chyba pomyliłeś linijkę - jest dodane jedynie linijkę niżej, w placeholderze. Przydało by się też dodać do opisu

@karolba
Copy link
Contributor

karolba commented Aug 8, 2022

Odpowiedź do #26 (comment)

Dodatkowo dodał bym check czy tekst do którego dopisujemy @student.pwr.edu.pl rzeczywiście jest indeksem.
Indeksy które obsługujemy to liczby (możliwie zaczynające się od zera) zawsze sześcio- lub pięciocyfrowe

Są też 4 cyfrowe. Wystarczy sprawdzanie, czy jest liczbą moim zdaniem.

Serio są cztero? Wiem że są doktorantów, ale od KWSu (czyli pewnie samych studentów) jedyne jakie widziałem najmniejsze to te kilka pięciocyfrowych

O doktorantów mi chodziło właśnie.

A doktorantów nie obsługujemy

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants