Skip to content

Conversation

petermasking
Copy link
Member

Fixes #361
Fixes #368

Changes proposed in this pull request:

  • Jitar v0.9
  • Resources
  • Index files

@MaskingTechnology/comify

@petermasking petermasking linked an issue Jan 10, 2025 that may be closed by this pull request
@@ -1,10 +1,10 @@

import getImageData from '^/domain/image/getById/feature';
import getImageData from '^/domain/image/getById/getById';
Copy link
Member

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,13 +1,13 @@

import createImage from '^/domain/image/create/feature';
import createImage from '^/domain/image/create/create';
Copy link
Member

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';
Copy link
Member

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';
Copy link
Member

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

Copy link
Member Author

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?

Copy link
Member

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.

@@ -0,0 +1,13 @@

import save from '../save/save';
Copy link
Member

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';
Copy link
Member

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';
Copy link
Member

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';
Copy link
Member

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.

@@ -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';
Copy link
Member

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';
Copy link
Member

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

basmasking
basmasking previously approved these changes Jan 10, 2025
Copy link

@basmasking basmasking merged commit c6f37f2 into main Jan 10, 2025
5 checks passed
@basmasking basmasking deleted the 368-upgrade-to-jitar-v09 branch January 10, 2025 22:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade to Jitar v0.9 Rewrite application for the next version of Jitar
2 participants