Let's have a look at the fixes done as follows:
- Fix—use a wrapped ID instead of a native ID in BoardMember: In com.taskagile.domain.model.board.BoardMember, getUserId() and getBoardId() 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 and RegisterPage.spec.js, besides i18n, a mocks 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