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:
![](https://static.packt-cdn.com/products/9781788836968/graphics/e1aa7d1a-8747-48a1-90b4-e0c4fc0a29bf.png)
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:
![](https://static.packt-cdn.com/products/9781788836968/graphics/a60d17fa-75ec-4178-828c-db5f6a717796.png)
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
:
![](https://static.packt-cdn.com/products/9781788836968/graphics/cc199338-29fd-47ee-bbdc-6210118c6978.png)
Figure 11.16: Fix: page header logo image not showing up