modulehandbook / studyplan

A MEVN-based Tool for HTW IMI students to plan their studies and choose optional modules.
GNU General Public License v3.0
5 stars 2 forks source link

Umfrage: Front-end logik überarbeiten #86

Closed BenSauerlaender closed 2 years ago

BenSauerlaender commented 2 years ago

Aktuelle Probleme:

Devianirfan commented 2 years ago

update: pushed to the branch I86 in Github. In this branch, user is able to send multiple data for Umfrage using check-box. But for the sonstiges part, still don't know how to handle it.

BenSauerlaender commented 2 years ago

Feedback for I86_Frontend_Umfrage_logik_ueberarbeiten:

First of all: Dont use hard-coded courses:

The survey data should not be stored anymore in courseSelection:

To fix other: Just dont use the same variable for the text-input-field AND to store the selected reason.

I think you don't need to use all the event stuff.

LennoxCode commented 2 years ago

Die Radio Buttons wurden durch checkboxes ersetzt, die es moeglich machen eine permutation aus allen gruenden zu waehlen und zu speichern. auch die moeglichkeit sonstige auszufuellen und zu speichern wurde ueberarbeitet. wenn die checkbox daneben gesetzt ist kann diese ausgefuellt werden und wird beim speichern beruecksichtigt. wenn sie erneut getoggled wird wird der inhalt gecleared