Fixes at the end
Let's have a look at the fixes done as follows:
- Fix—use a wrapped ID instead of a native ID in
BoardMember
: Incom.taskagile.domain.model.board.BoardMember
,getUserId()
andgetBoardId()
return a native long value. It is better to use wrapped IDs:
Figure 11.14: Fix: use wrapped id instead of native id in BoardMember commit
- Fix—remove unnecessary mocks: In
LoginPage.spec.js
andRegisterPage.spec.js
, besidesi18n
, amocks
object for$t
is also added to the wrapper, which is unnecessary. It is better to remove it to avoid confusion:
Figure 11.15: Fix: remove unnecessary mocks commit
- Page header logo image not showing up: In
PageHeader.vue
, the logo image path is still/static/images/logo.png
. It should be changed to/images/logo.png
:
Figure 11.16: Fix: page header logo image not showing up