Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feature/333 tests #363

Merged
merged 160 commits into from
Mar 16, 2022
Merged

Feature/333 tests #363

merged 160 commits into from
Mar 16, 2022

Conversation

mburri
Copy link
Contributor

@mburri mburri commented Mar 14, 2022

Umsetzung der Übersichtsseite für Tests.

Die Noten werden fix angezeigt - es gibt noch keine Inputs (Punkteeingabe) bzw. Dropdown für die Noten - das wird mit Story #336 umgesetzt
Ebenfalls noch nicht angezeigt werden die Mittelwerte/ bzw. es werden hart codierte Werte angezeigt - korrekte Anzeige/ Werte wird mit #338 umgesetzt

caebr and others added 30 commits February 9, 2022 16:43
@caebr
add new module events, refs 332
a6ba8f8
@caebr
add request for formative assessments, refs 332
79a6260
@caebr
update study class model
2f593e4
@caebr
add course model
4cc88c6
@caebr
remove empty lifecycle method
0998f3a
@caebr
add courses rest service and implement expanded courses request, refs…
c2711c3 
… 332
@mfehlmann
Generate module tests refs: 333
e40c9e9
@mfehlmann
Remove e2e from linter config
7c57ae8
@mfehlmann
Generate component tests refs: 333
0d8408b
@mfehlmann
Routing for tests refs: 333
6c4377f
@mfehlmann
Move tests component under components, refs 333
9acb40f
@mfehlmann
move tests component under components in events module refs: 333
37679bb
@mburri
remove tests entry point from home component, refs 333
f27796e
@mburri
setup route to tests with a paath param, refs 333
ab59aa5
@mburri
add component to show list of tests, refs 333
42ccce6
@mburri
get course id as observable from path param, refs 333
6d50a2f
@mburri
remove unused router, refs 333
1fbec73
@mburri
correct setup for tests and inject router
cb9e9a0
@mburri
move test-list component to components folder, refs 333
4b96d33
@mburri
rename events component to courses component - but keep it in the eve…
9b24cc6 
…nts module, refs 333
@mfehlmann
Remove tests from home component (refs: 333)
e3ea1e8
@caebr
remove accidentially added fdescribe, refs 333
cad6663
@caebr
add courses list component, refs 332
f688710
@caebr
fix course model expanded references, refs 332
6674dfd
@mburri
load course with test from endpoint and show course designation, refs…
74fa4b6 
… 333
@mburri
add convinient link to open a course with tests, refs 333
6ce679d
@mburri
link to course without tests, refs 333
5e53708
@mburri
show hint to create new test if no tests are available in course, ref…
c1643aa 
…s 333
@caebr
update link to nglet directive
f4ab970
@caebr
add courses state service and show course designation on list compone…
5fe8731 
…nt, refs 332
mburri and others added 11 commits March 14, 2022 10:00
@mburri
ellipsis for test owner, align dates on same line vertically, refs 333
54fb751
@mburri
show edit icon when test is not published and the current user is the…
53e2144 
… test owner, refs 333
@mburri
fix points/ grade column width, refs 333
52417a2
@mburri
hide test designation on mobile, refs 333
9cfa377
@mburri
show expand toggle for mobile, refs 333
e4c19e3
@mburri
remove fdescribe
0991a3a
@mburri
abbreviate test designation if collapsed, refs 333
7e479c5
@mburri
use ellipsis for test designation if not expanded, refs 333
0426ec2
@mburri
fix width of table cell with test info for mobile, refs 333
ce4e562
@caebr @mburri
Add status id filter and teacher role restriction to courses request,…
a65f957 
… refs 332
@mburri
Merge pull request #362 from bkd-mba-fbi/feature/332-course-request
7a56b5a 
Add status id filter and teacher role restriction to courses request,…
@mburri mburri requested a review from caebr March 14, 2022 13:47
@caebr
Copy link
Collaborator

caebr commented Mar 14, 2022 •
edited by mburri
Loading

Es hat noch einige Layout-Details die nicht stimmen. Ich habe angefange sie zu korrigieren, bin aber nicht fertig geworden. Hier eine Liste, zur Erinnerung:

  • Table sollte "responsive" sein, also horizontal scrollen. Aber nur ab der vierten Spalte, wenn ich das richtig interpretiere auf dem Mockup.
  • Accordion: Cursor sollte Pointer sein
  • Erste drei Spalten sollten fix sein; nicht springen beim Klicken vom "Alle" "Meine" Tests
  • Sollten die Namen nicht ellipsiert werden wenn lang?
  • Text Buttons und Name des Tests sind nicht aligned
  • In Arbeit sollte blau sein
  • Wieso ist der Edit-Button beim Datum?
  • Das Responsive Layout bricht bei ca. 750 um aber das Dropdown kommt erst ab ca. 600
  • Unnötige horizontale Linie nach Dropdown
  • In der mobile Version stimmt der Abstand link und recht nicht (nicht wie andere Module)
  • In der mobile Version springt das Caret herum, wenn man auf- und zuklapp

@mburri
Copy link
Contributor Author

mburri commented Mar 15, 2022

  • dropdown zur Auswahl des Tests hat einen anderen Breakpoint als die Tabelle...

@mburri mburri merged commit d06a9db into master Mar 16, 2022
@mfehlmann mfehlmann deleted the feature/333-tests branch May 9, 2022 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@caebr caebr Awaiting requested review from caebr

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mburri @caebr @mfehlmann