-
Notifications
You must be signed in to change notification settings - Fork 3
368 upgrade to jitar v09 #369
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@@ -1,10 +1,10 @@ | |||
|
|||
import getImageData from '^/domain/image/getById/feature'; | |||
import getImageData from '^/domain/image/getById/getById'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Imports directly, instead of using index file
src/domain/comic/create/create.ts
Outdated
@@ -1,13 +1,13 @@ | |||
|
|||
import createImage from '^/domain/image/create/feature'; | |||
import createImage from '^/domain/image/create/create'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Imports directly, instead of using index file
@@ -1,10 +1,10 @@ | |||
|
|||
import getImageData from '^/domain/image/getById/feature'; | |||
import getImageData from '^/domain/image/getById/getById'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Imports directly, instead of using index file
@@ -1,5 +1,5 @@ | |||
|
|||
import downloadImage from '^/domain/image/download/feature'; | |||
import downloadImage from '^/domain/image/download'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Imports directly, instead of using index file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Imports from the feature's index file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I must be mistaken, but this is indeed correct.
src/domain/image/create/create.ts
Outdated
@@ -0,0 +1,13 @@ | |||
|
|||
import save from '../save/save'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Imports directly, instead of using index file
@@ -0,0 +1,12 @@ | |||
|
|||
import { Requester } from '^/domain/authentication'; | |||
import createComic from '^/domain/comic/create/create'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Imports directly, instead of using index file
@@ -1,5 +1,5 @@ | |||
|
|||
import database from '^/integrations/database/module'; | |||
import database from '^/integrations/database/index.js'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
index.js ?
import updateNickname from '^/domain/creator/updateNickname/feature'; | ||
import { requester } from '^/domain/authentication'; | ||
import type { AggregatedData as AggregatedCreatorData } from '^/domain/creator/aggregate'; | ||
//import updateNickname, { NicknameAlreadyExists } from '^/domain/creator/updateNickname'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comment doesn't seem deleted.
test/domain/comic/create.spec.ts
Outdated
@@ -1,10 +1,10 @@ | |||
|
|||
import { beforeEach, describe, expect, it } from 'vitest'; | |||
|
|||
import create from '^/domain/comic/create/feature'; | |||
import create from '^/domain/comic/create/create'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Imports directly, instead of using index file
@@ -1,12 +1,12 @@ | |||
|
|||
import johnDoe from '^/domain/authentication/requester'; | |||
import { requester } from '^/domain/authentication'; | |||
import { Requester } from '^/domain/authentication/types'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Imports directly, instead of using index file
|
Fixes #361
Fixes #368
Changes proposed in this pull request:
@MaskingTechnology/comify