Devoxx4Kids-NPO / littil-frontend

Apache License 2.0
0 stars 4 forks source link

Add error message to form base + split ApiService into separate services #22

Closed Anjanas-dh closed 2 years ago

Anjanas-dh commented 2 years ago

Before refactoring for oauth: merge master, fix unit tests and add error message to form base

github-actions[bot] commented 2 years ago

Coverage report

St.:grey_question:
Category Percentage Covered / Total
🟒 Statements
84.88% (-8.37% πŸ”»)
146/172
🟑 Branches
76.67% (+24.67% πŸ”Ό)
23/30
🟒 Functions
83.72% (-4.51% πŸ”»)
36/43
🟒 Lines
82.07% (-9.71% πŸ”»)
119/145
Show new covered files 🐣
|
St.:grey_question:
| File | Statements | Branches | Functions | Lines | | :----------------------------------------------------------------------------------------------------------------------------------------------------: | :----------------------------------------------------------------------------------------------------- | :--------- | :------- | :-------- | :----- | | 🟒 |
`...` / login-modal.component.ts
| 100% | 100% | 100% | 100% | | 🟒 |
`...` / login-modal.component.html
| 100% | 100% | 100% | 100% | | 🟒 |
`...` / littil-teacher.service.ts
| 100% | 100% | 100% | 100% | | 🟒 |
`...` / littil-school.service.ts
| 100% | 100% | 100% | 100% | | πŸ”΄ |
`...` / modal.controller.ts
| 25.71% | 0% | 12.5% | 18.75% | | 🟒 |
`...` / homepage.component.ts
| 100% | 100% | 100% | 100% | | 🟒 |
`...` / homepage.component.html
| 100% | 100% | 100% | 100% |

Test suite run success

87 tests passing in 13 suites.

Report generated by πŸ§ͺjest coverage report action from adc3a5120890211623d5a4a2c7b3927c034eb230

Anjanas-dh commented 2 years ago

Ik heb de ApiService weer gesplitst: separation of concerns (LittilTeacherService + LittilSchoolService) De loginModal is met het oog op de refactoring voor de oAuthflow nu puur een login formulier. Register wordt een aparte modal. En er komt een nieuwe flow bij: finishRegistrationModal - hier kunnen users aangeven of ze een school zijn of een guestTeacher die les wilt gaan geven en al hun gegevens opgeven (Role zetten)

Anjanas-dh commented 2 years ago

Ik merge hem voor nu, zodat Arjen en Marijn ook verder kunnen. Als er nog wijzigingen uit de comments komen, doe ik dat in een aparte PR @LDMGN