s426135
/
session-companion
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

SES-69 Added controllers and comments #17

Merged
s426135 merged 2 commits from SES-69 into master 2020-12-06 15:29:13 +01:00
Conversation 1 Commits 2 Files Changed 9 +152 -20
s426134 commented 2020-12-06 14:37:43 +01:00
Collaborator
Copy Link

Dodałem komentarze do kiku plików.
Ogólnie zamysła jest taki ze komentujemy metody wystawione na front i viewmodele. Tak żeby ktoś mógł zoabczyć na swaggerze tylko co oznacza jakie pole i tyle, bez zbędnego pytania.
Ułatwi to nam pracę.
Żeby zachować czystość kodu dałem kontrolery bez metod. Nie ma sensyw ystawiać metod, których nie użyjemy nigdy.
Zastanawiam się nad komentowaniem metod w serwisach, co sądzisz o tym ?

Dodałem komentarze do kiku plików. Ogólnie zamysła jest taki ze komentujemy metody wystawione na front i viewmodele. Tak żeby ktoś mógł zoabczyć na swaggerze tylko co oznacza jakie pole i tyle, bez zbędnego pytania. Ułatwi to nam pracę. Żeby zachować czystość kodu dałem kontrolery bez metod. Nie ma sensyw ystawiać metod, których nie użyjemy nigdy. Zastanawiam się nad komentowaniem metod w serwisach, co sądzisz o tym ?
s426135 was assigned by s426134 2020-12-06 14:37:43 +01:00
s426135 commented 2020-12-06 15:27:03 +01:00
Owner
Copy Link

Ta dobry pomysł. Komentarze nigdy nie bolą xd

Ta dobry pomysł. Komentarze nigdy nie bolą xd
s426135 approved these changes 2020-12-06 15:28:16 +01:00
s426135 left a comment
Owner
Copy Link

Okej wygląda dobrze

Okej wygląda dobrze
s426135 merged commit ccfdff6791 into master 2020-12-06 15:29:13 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
s426135
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
s426135
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: s426135/session-companion#17
No description provided.
Delete Branch "SES-69"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?