Skip to content

feature/nft-domain-implement #192

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.

Sign up for GitHub

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

Open
JXPlayer wants to merge 1 commit into
base: main
Choose a base branch
from
Open

feature/nft-domain-implement #192

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions src/modules/nft/domain/exceptions/ntf.exceptions.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
import { DomainException } from "@/modules/shared/domain/exceptions/domain.exception";


export class NTFException extends DomainException {
Copy link

@coderabbitai coderabbitai bot Sep 1, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Verification agent

🧩 Analysis chain

Fix typo: use NFTException and align filename.

“NTF” looks like a typo. Please rename the class to NFTException and rename the file to nft.exceptions.ts to keep the acronym consistent and avoid confusing imports/autocomplete.

Apply this diff:

-export class NTFException extends DomainException {
+export class NFTException extends DomainException {

Run to find any stray NTF/NFT usages and filename inconsistencies:

🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "== Occurrences of NTFException / NFTException =="
rg -nP '\bNTFException\b|\bNFTException\b' -g '!**/node_modules/**' -C2 || true

echo "== Files named *ntf*.exceptions.ts vs *nft*.exceptions.ts =="
fd -a -i 'ntf.exceptions.ts' || true
fd -a -i 'nft.exceptions.ts' || true

Length of output: 837

Rename class and file to correct acronym
Rename src/modules/nft/domain/exceptions/ntf.exceptions.ts → nft.exceptions.ts and update the class name:

--- a/src/modules/nft/domain/exceptions/ntf.exceptions.ts
+++ b/src/modules/nft/domain/exceptions/nft.exceptions.ts
-export class NTFException extends DomainException {
+export class NFTException extends DomainException {

Update any imports accordingly (no other occurrences detected).

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
export class NTFException extends DomainException {
export class NFTException extends DomainException {
🤖 Prompt for AI Agents 
In src/modules/nft/domain/exceptions/ntf.exceptions.ts around line 4, the file
and exported class use the incorrect acronym "NTF"; rename the file to
src/modules/nft/domain/exceptions/nft.exceptions.ts and rename the exported
class from NTFException to NFTException, update its filename-based export
accordingly, and update all imports across the codebase to reference the new
filename and class name.

constructor(message: string) {
super(message);
}
}