search

coders-school / desktop-business-app

0 stars 4 forks source link

37 create reserve queue #69

Closed mBialczak closed 6 months ago

mBialczak commented 6 months ago

Introduced ReserveQueue class.

Most of function names have new proposal for names (picked carefully IMHO) differ when compared to current documentation. If names are accepted and PR is merged - I will update Docs accordingly.

After some thought I decided that makeAppointment() from docs makes no sense in this class. It would be Single Responsibility rule violation - it is not up to ResevrveQueue to make new Visists. Other classes should be/are responsible for new visit creations.

Some minor corrections to CMakeLists.txt files introduced with minor fixes of fixed include paths.

Introduced test utils with one function so far - cleanupClinic().

ziobron commented 6 months ago

Thanks @fbtom for Code Review. 🏅 1 XP granted

ziobron commented 6 months ago

Thanks @fbtom for Code Review. 🏅 1 XP granted

ziobron commented 6 months ago

Thanks @fbtom for Code Review. 🏅 1 XP granted

ziobron commented 6 months ago

Thanks @fbtom for Code Review. 🏅 1 XP granted

ziobron commented 6 months ago

Your PR was merged! 🏅 2 XP granted. Thanks @mBialczak for making course materials better!